From 610a09ccff8935996568539ba5bdd92a65e49743 Mon Sep 17 00:00:00 2001 From: anand76 Date: Wed, 15 Apr 2020 12:09:33 -0700 Subject: [PATCH] Remove a printf from db_stress that's not useful info (#6705) Summary: This was causing db_crashtest.py to wrongly assume an error by parsing the output. Hopefully this will stabilize the crash tests. Pull Request resolved: https://github.com/facebook/rocksdb/pull/6705 Test Plan: make blackbox_crash_test Reviewed By: ltamasi Differential Revision: D21043335 Pulled By: anand1976 fbshipit-source-id: 5cddd112b124d4e2ebd11724a17d4ef0f50c1cf8 --- db_stress_tool/db_stress_stat.h | 2 -- tools/db_crashtest.py | 3 +-- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/db_stress_tool/db_stress_stat.h b/db_stress_tool/db_stress_stat.h index a0accc4d9..e8bc0986a 100644 --- a/db_stress_tool/db_stress_stat.h +++ b/db_stress_tool/db_stress_stat.h @@ -204,8 +204,6 @@ class Stats { covered_by_range_deletions_); fprintf(stdout, "%-12s: Got errors %ld times\n", "", errors_); - fprintf(stdout, "%-12s: Got expected errors %ld times\n", "", - verified_errors_); fprintf(stdout, "%-12s: %ld CompactFiles() succeed\n", "", num_compact_files_succeed_); fprintf(stdout, "%-12s: %ld CompactFiles() did not succeed\n", "", diff --git a/tools/db_crashtest.py b/tools/db_crashtest.py index d824cc64c..e947bf604 100644 --- a/tools/db_crashtest.py +++ b/tools/db_crashtest.py @@ -453,8 +453,7 @@ def whitebox_crash_main(args, unknown_args): stdoutdata = stdoutdata.lower() errorcount = (stdoutdata.count('error') - - stdoutdata.count('got errors 0 times') - - stdoutdata.count('got expected errors 0 times')) + stdoutdata.count('got errors 0 times')) print("#times error occurred in output is " + str(errorcount) + "\n") if (errorcount > 0):