add fix

apache · Mar 1, 2023 · e98868c · e98868c
1 parent ece6cc9
commit e98868c
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/test/bench_test/benchmark.cpp b/src/test/bench_test/benchmark.cpp
@@ -51,7 +51,10 @@ DSN_DEFINE_int32(pegasus.benchmark, multi_count, 100, "Values count of the same
 DSN_DEFINE_string(pegasus.benchmark, benchmarks, "", "method by operation type");
 
 DSN_DEFINE_group_validator(multi_count, [](std::string &message) -> bool {
-    if ((FLAGS_benchmarks == "multisetrandom_pegasus" || FLAGS_benchmarks == "multigetrandom_pegasus") && FLAGS_benchmark_num % FLAGS_multi_count != 0) {
+    std::string operation_type = FLAGS_benchmarks;
+    if ((operation_type == "multisetrandom_pegasus" ||
+         operation_type == "multigetrandom_pegasus") &&
+        FLAGS_benchmark_num % FLAGS_multi_count != 0) {
         message = fmt::format("num {} should be a multiple of multi_count({}).",
                               FLAGS_benchmark_num,
                               FLAGS_multi_count);
@@ -198,7 +201,6 @@ void benchmark::multi_set_random(thread_arg *thread)
                 dsn_exit(1);
             }
             fmt::print(stderr, "Set timeout, retry({})\n", try_count);
-
         }
 
         // Count this operation.