update delete benchmarks

gofiber · Sep 15, 2023 · db1dd2b · db1dd2b
1 parent 36dd674
commit db1dd2b
Show file tree

Hide file tree

Showing 21 changed files with 109 additions and 93 deletions.
diff --git a/arangodb/arangodb_test.go b/arangodb/arangodb_test.go
@@ -158,14 +158,13 @@ func Benchmark_ArangoDB_Get(b *testing.B) {
 	require.NoError(b, err)
 }
 
-func Benchmark_ArangoDB_Delete(b *testing.B) {
-	err := testStore.Set("john", []byte("doe"), 0)
-	require.NoError(b, err)
-
+func Benchmark_ArangoDB_SetAndDelete(b *testing.B) {
 	b.ReportAllocs()
 	b.ResetTimer()
 
+	var err error
 	for i := 0; i < b.N; i++ {
+		_ = testStore.Set("john", []byte("doe"), 0)
 		err = testStore.Delete("john")
 	}
 

diff --git a/azureblob/azureblob_test.go b/azureblob/azureblob_test.go
@@ -163,14 +163,13 @@ func Benchmark_AzureBlob_Get(b *testing.B) {
 	require.NoError(b, err)
 }
 
-func Benchmark_AzureBlob_Delete(b *testing.B) {
-	err := testStore.Set("john", []byte("doe"), 0)
-	require.NoError(b, err)
-
+func Benchmark_AzureBlob_SetAndDelete(b *testing.B) {
 	b.ReportAllocs()
 	b.ResetTimer()
 
+	var err error
 	for i := 0; i < b.N; i++ {
+		_ = testStore.Set("john", []byte("doe"), 0)
 		err = testStore.Delete("john")
 	}
 

diff --git a/badger/badger_test.go b/badger/badger_test.go
@@ -148,13 +148,12 @@ func Benchmark_Badger_Get(b *testing.B) {
 }
 
 func Benchmark_Badger_Delete(b *testing.B) {
-	err := testStore.Set("john", []byte("doe"), 0)
-	require.NoError(b, err)
-
 	b.ReportAllocs()
 	b.ResetTimer()
 
+	var err error
 	for i := 0; i < b.N; i++ {
+		_ = testStore.Set("john", []byte("doe"), 0)
 		err = testStore.Delete("john")
 	}
 

diff --git a/bbolt/bbolt_test.go b/bbolt/bbolt_test.go
@@ -137,13 +137,12 @@ func Benchmark_Bbolt_Get(b *testing.B) {
 }
 
 func Benchmark_Bbolt_Delete(b *testing.B) {
-	err := testStore.Set("john", []byte("doe"), 0)
-	require.NoError(b, err)
-
 	b.ReportAllocs()
 	b.ResetTimer()
 
+	var err error
 	for i := 0; i < b.N; i++ {
+		_ = testStore.Set("john", []byte("doe"), 0)
 		err = testStore.Delete("john")
 	}
 

diff --git a/coherence/coherence_test.go b/coherence/coherence_test.go
@@ -216,14 +216,13 @@ func Benchmark_Coherence_Get(b *testing.B) {
 	require.NoError(b, err)
 }
 
-func Benchmark_Coherence_Delete(b *testing.B) {
-	err := testStore.Set("john", []byte("doe"), 0)
-	require.NoError(b, err)
-
+func Benchmark_Coherence_SetAndDelete(b *testing.B) {
 	b.ReportAllocs()
 	b.ResetTimer()
 
+	var err error
 	for i := 0; i < b.N; i++ {
+		_ = testStore.Set("john", []byte("doe"), 0)
 		err = testStore.Delete("john")
 	}
 

diff --git a/couchbase/couchbase_test.go b/couchbase/couchbase_test.go
@@ -129,16 +129,15 @@ func Benchmark_Couchbase_Get(b *testing.B) {
 	require.NoError(b, err)
 }
 
-func Benchmark_Couchbase_Delete(b *testing.B) {
+func Benchmark_Couchbase_SetAndDelete(b *testing.B) {
 	testStore := newTestStore(b)
 
-	err := testStore.Set("john", []byte("doe"), 0)
-	require.NoError(b, err)
-
 	b.ReportAllocs()
 	b.ResetTimer()
 
+	var err error
 	for i := 0; i < b.N; i++ {
+		_ = testStore.Set("john", []byte("doe"), 0)
 		err = testStore.Delete("john")
 	}
 

diff --git a/dynamodb/dynamodb_test.go b/dynamodb/dynamodb_test.go
@@ -115,3 +115,42 @@ func Test_DynamoDB_Close(t *testing.T) {
 func Test_DynamoDB_Conn(t *testing.T) {
 	require.True(t, testStore.Conn() != nil)
 }
+
+func Benchmark_DynamoDB_Set(b *testing.B) {
+	b.ReportAllocs()
+	b.ResetTimer()
+
+	var err error
+	for i := 0; i < b.N; i++ {
+		err = testStore.Set("john", []byte("doe"), 0)
+	}
+
+	require.NoError(b, err)
+}
+
+func Benchmark_DynamoDB_Get(b *testing.B) {
+	err := testStore.Set("john", []byte("doe"), 0)
+	require.NoError(b, err)
+
+	b.ReportAllocs()
+	b.ResetTimer()
+
+	for i := 0; i < b.N; i++ {
+		_, err = testStore.Get("john")
+	}
+
+	require.NoError(b, err)
+}
+
+func Benchmark_DynamoDB_SetAndDelete(b *testing.B) {
+	b.ReportAllocs()
+	b.ResetTimer()
+
+	var err error
+	for i := 0; i < b.N; i++ {
+		_ = testStore.Set("john", []byte("doe"), 0)
+		err = testStore.Delete("john")
+	}
+
+	require.NoError(b, err)
+}
diff --git a/etcd/etcd_test.go b/etcd/etcd_test.go
@@ -118,14 +118,13 @@ func Benchmark_Etcd_Get(b *testing.B) {
 	require.NoError(b, err)
 }
 
-func Benchmark_Etcd_Delete(b *testing.B) {
-	err := testStore.Set("john", []byte("doe"), 0)
-	require.NoError(b, err)
-
+func Benchmark_Etcd_SetAndDelete(b *testing.B) {
 	b.ReportAllocs()
 	b.ResetTimer()
 
+	var err error
 	for i := 0; i < b.N; i++ {
+		_ = testStore.Set("john", []byte("doe"), 0)
 		err = testStore.Delete("john")
 	}
 

diff --git a/memcache/memcache_test.go b/memcache/memcache_test.go
@@ -157,14 +157,13 @@ func Benchmark_Memcache_Get(b *testing.B) {
 	require.NoError(b, err)
 }
 
-func Benchmark_Memcache_Delete(b *testing.B) {
-	err := testStore.Set("john", []byte("doe"), 0)
-	require.NoError(b, err)
-
+func Benchmark_Memcache_SetAndDelete(b *testing.B) {
 	b.ReportAllocs()
 	b.ResetTimer()
 
+	var err error
 	for i := 0; i < b.N; i++ {
+		_ = testStore.Set("john", []byte("doe"), 0)
 		err = testStore.Delete("john")
 	}
 

diff --git a/memory/memory_test.go b/memory/memory_test.go
@@ -174,14 +174,13 @@ func Benchmark_Memory_Get(b *testing.B) {
 	require.NoError(b, err)
 }
 
-func Benchmark_Memory_Delete(b *testing.B) {
-	err := testStore.Set("john", []byte("doe"), 0)
-	require.NoError(b, err)
-
+func Benchmark_Memory_SetAndDelete(b *testing.B) {
 	b.ReportAllocs()
 	b.ResetTimer()
 
+	var err error
 	for i := 0; i < b.N; i++ {
+		_ = testStore.Set("john", []byte("doe"), 0)
 		err = testStore.Delete("john")
 	}
 

diff --git a/minio/minio_test.go b/minio/minio_test.go
@@ -220,14 +220,13 @@ func Benchmark_Minio_Get(b *testing.B) {
 	require.NoError(b, err)
 }
 
-func Benchmark_Minio_Delete(b *testing.B) {
-	err := testStore.Set("john", []byte("doe"), 0)
-	require.NoError(b, err)
-
+func Benchmark_Minio_SetAndDelete(b *testing.B) {
 	b.ReportAllocs()
 	b.ResetTimer()
 
+	var err error
 	for i := 0; i < b.N; i++ {
+		_ = testStore.Set("john", []byte("doe"), 0)
 		err = testStore.Delete("john")
 	}
 

diff --git a/mongodb/mongodb_test.go b/mongodb/mongodb_test.go
@@ -157,14 +157,13 @@ func Benchmark_MongoDB_Get(b *testing.B) {
 	require.NoError(b, err)
 }
 
-func Benchmark_MongoDB_Delete(b *testing.B) {
-	err := testStore.Set("john", []byte("doe"), 0)
-	require.NoError(b, err)
-
+func Benchmark_MongoDB_SetAndDelete(b *testing.B) {
 	b.ReportAllocs()
 	b.ResetTimer()
 
+	var err error
 	for i := 0; i < b.N; i++ {
+		_ = testStore.Set("john", []byte("doe"), 0)
 		err = testStore.Delete("john")
 	}
 

diff --git a/mssql/mssql_test.go b/mssql/mssql_test.go
@@ -200,14 +200,13 @@ func Benchmark_MSSQL_Get(b *testing.B) {
 	require.NoError(b, err)
 }
 
-func Benchmark_MSSQL_Delete(b *testing.B) {
-	err := testStore.Set("john", []byte("doe"), 0)
-	require.NoError(b, err)
-
+func Benchmark_MSSQL_SetAndDelete(b *testing.B) {
 	b.ReportAllocs()
 	b.ResetTimer()
 
+	var err error
 	for i := 0; i < b.N; i++ {
+		_ = testStore.Set("john", []byte("doe"), 0)
 		err = testStore.Delete("john")
 	}
 

diff --git a/mysql/mysql_test.go b/mysql/mysql_test.go
@@ -216,14 +216,13 @@ func Benchmark_MYSQL_Get(b *testing.B) {
 	require.NoError(b, err)
 }
 
-func Benchmark_MYSQL_Delete(b *testing.B) {
-	err := testStore.Set("john", []byte("doe"), 0)
-	require.NoError(b, err)
-
+func Benchmark_MYSQL_SetAndDelete(b *testing.B) {
 	b.ReportAllocs()
 	b.ResetTimer()
 
+	var err error
 	for i := 0; i < b.N; i++ {
+		_ = testStore.Set("john", []byte("doe"), 0)
 		err = testStore.Delete("john")
 	}
 

diff --git a/pebble/pebble_test.go b/pebble/pebble_test.go
@@ -132,14 +132,13 @@ func Benchmark_Pebble_Get(b *testing.B) {
 	require.NoError(b, err)
 }
 
-func Benchmark_Pebble_Delete(b *testing.B) {
-	err := testStore.Set("john", []byte("doe"), 0)
-	require.NoError(b, err)
-
+func Benchmark_Pebble_SetAndDelete(b *testing.B) {
 	b.ReportAllocs()
 	b.ResetTimer()
 
+	var err error
 	for i := 0; i < b.N; i++ {
+		_ = testStore.Set("john", []byte("doe"), 0)
 		err = testStore.Delete("john")
 	}
 

diff --git a/postgres/postgres_test.go b/postgres/postgres_test.go
@@ -197,14 +197,13 @@ func Benchmark_Postgres_Get(b *testing.B) {
 	require.NoError(b, err)
 }
 
-func Benchmark_Postgres_Delete(b *testing.B) {
-	err := testStore.Set("john", []byte("doe"), 0)
-	require.NoError(b, err)
-
+func Benchmark_Postgres_SetAndDelete(b *testing.B) {
 	b.ReportAllocs()
 	b.ResetTimer()
 
+	var err error
 	for i := 0; i < b.N; i++ {
+		_ = testStore.Set("john", []byte("doe"), 0)
 		err = testStore.Delete("john")
 	}
 

diff --git a/redis/redis_test.go b/redis/redis_test.go
@@ -152,11 +152,10 @@ func Test_Redis_Initalize_WithURL_TLS(t *testing.T) {
 		return
 	}
 	tlsCfg := &tls.Config{
-		MinVersion:               tls.VersionTLS12,
-		CurvePreferences:         []tls.CurveID{tls.CurveP521, tls.CurveP384, tls.CurveP256},
-		PreferServerCipherSuites: true,
-		InsecureSkipVerify:       true,
-		Certificates:             []tls.Certificate{cer},
+		MinVersion:         tls.VersionTLS12,
+		CurvePreferences:   []tls.CurveID{tls.CurveP521, tls.CurveP384, tls.CurveP256},
+		InsecureSkipVerify: true,
+		Certificates:       []tls.Certificate{cer},
 		CipherSuites: []uint16{
 			tls.TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384,
 			tls.TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA,
@@ -195,11 +194,10 @@ func Test_Redis_Initalize_WithURL_TLS_Verify(t *testing.T) {
 		return
 	}
 	tlsCfg := &tls.Config{
-		MinVersion:               tls.VersionTLS12,
-		CurvePreferences:         []tls.CurveID{tls.CurveP521, tls.CurveP384, tls.CurveP256},
-		PreferServerCipherSuites: true,
-		InsecureSkipVerify:       false,
-		Certificates:             []tls.Certificate{cer},
+		MinVersion:         tls.VersionTLS12,
+		CurvePreferences:   []tls.CurveID{tls.CurveP521, tls.CurveP384, tls.CurveP256},
+		InsecureSkipVerify: false,
+		Certificates:       []tls.Certificate{cer},
 		CipherSuites: []uint16{
 			tls.TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384,
 			tls.TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA,
@@ -441,14 +439,13 @@ func Benchmark_Redis_Get(b *testing.B) {
 	require.NoError(b, err)
 }
 
-func Benchmark_Redis_Delete(b *testing.B) {
-	err := testStore.Set("john", []byte("doe"), 0)
-	require.NoError(b, err)
-
+func Benchmark_Redis_SetAndDelete(b *testing.B) {
 	b.ReportAllocs()
 	b.ResetTimer()
 
+	var err error
 	for i := 0; i < b.N; i++ {
+		_ = testStore.Set("john", []byte("doe"), 0)
 		err = testStore.Delete("john")
 	}
 

diff --git a/ristretto/ristretto_test.go b/ristretto/ristretto_test.go
@@ -84,7 +84,7 @@ func Test_Ristretto_Set_Expiration(t *testing.T) {
 	err := testStore.Set(key, val, exp)
 	require.NoError(t, err)
 
-	testStore.Reset()
+	require.NoError(t, testStore.Reset())
 }
 
 func Test_Ristretto_Get_Expired(t *testing.T) {
@@ -190,14 +190,13 @@ func Benchmark_Ristretto_Get(b *testing.B) {
 	require.NoError(b, err)
 }
 
-func Benchmark_Ristretto_Delete(b *testing.B) {
-	err := testStore.Set("john", []byte("doe"), 0)
-	require.NoError(b, err)
-
+func Benchmark_Ristretto_SetAndDelete(b *testing.B) {
 	b.ReportAllocs()
 	b.ResetTimer()
 
+	var err error
 	for i := 0; i < b.N; i++ {
+		_ = testStore.Set("john", []byte("doe"), 0)
 		err = testStore.Delete("john")
 	}