Skip to content
  • Pratik Dhandharia's avatar
    Lower the risk for users to run options.force_consistency_checks = true (#5744) · a2818223
    Pratik Dhandharia authored
    Summary:
    Open-source users recently reported two occurrences of LSM-tree corruption (https://github.com/facebook/rocksdb/issues/5558 is one), which would be caught by options.force_consistency_checks = true. options.force_consistency_checks has a usability limitation because it crashes the service once inconsistency is detected. This makes the feature hard to use. Most users serve from multiple RocksDB shards per server and the impacts of crashing the service is higher than it should be.
    
    Instead, we just pass the error back to users without killing the service, and ask them to deal with the problem accordingly.
    Pull Request resolved: https://github.com/facebook/rocksdb/pull/5744
    
    Differential Revision: D17096940
    
    Pulled By: pdhandharia
    
    fbshipit-source-id: b6780039044e265f26ed2ad03c51f4abbe8b603c
    a2818223