Revision 8e12638f3d0d91791cf06253493b8b15827f4b6c authored by sdong on 19 August 2019, 17:50:25 UTC, committed by Facebook Github Bot on 19 August 2019, 17:51:59 UTC
Summary: Atomic white box test's kill odd is the same as normal test. However, in the scenario that only WritableFileWriter::Append() is blacklisted, WritableFileWriter::Flush() dominates the killing odds. Normally, most of WritableFileWriter::Flush() are called in WAL writes, where every write triggers a WAL flush. In atomic test, WAL is disabled, so the kill happens less frequently than we antipated. In some rare cases, the kill didn't end up with happening (for reasons I still don't fully understand) and cause the stress test timeout. If WAL is disabled, make the odds 5x likely to trigger. Pull Request resolved: https://github.com/facebook/rocksdb/pull/5717 Test Plan: Run whitebox_crash_test_with_atomic_flush and whitebox_crash_test and observe the kill odds printed out. Differential Revision: D16897237 fbshipit-source-id: cbf5d96f6fc0e980523d0f1f94bf4e72cdb82d1c
1 parent e1c468d
| File | Mode | Size |
|---|---|---|
| db_impl.cc | -rw-r--r-- | 148.8 KB |
| db_impl.h | -rw-r--r-- | 83.0 KB |
| db_impl_compaction_flush.cc | -rw-r--r-- | 115.9 KB |
| db_impl_debug.cc | -rw-r--r-- | 8.5 KB |
| db_impl_experimental.cc | -rw-r--r-- | 5.4 KB |
| db_impl_files.cc | -rw-r--r-- | 24.0 KB |
| db_impl_open.cc | -rw-r--r-- | 55.3 KB |
| db_impl_readonly.cc | -rw-r--r-- | 7.8 KB |
| db_impl_readonly.h | -rw-r--r-- | 5.2 KB |
| db_impl_secondary.cc | -rw-r--r-- | 24.1 KB |
| db_impl_secondary.h | -rw-r--r-- | 11.4 KB |
| db_impl_write.cc | -rw-r--r-- | 65.7 KB |
| db_secondary_test.cc | -rw-r--r-- | 25.3 KB |
Computing file changes ...
