Pranav Shenoy

IsEmpty() checks were missing before some SwitchMemtable() invocations

1

Fixes #12179 There was one instance where the IsEmpty() check was missing: https://github.com/facebook/rocksdb/blob/21d5a8f54f06e01ca49d4e6bae89bb42ba78dfdd/db/db_impl/db_impl_write.cc#L1706 Having one check inside SwitchMemTable() will ensure there won't be misses in the future.

IsEmpty() checks were missing before some SwitchMemtable() invocation…

1

Bug: https://github.com/facebook/rocksdb/issues/12179 There was one instance where the IsEmpty() check was missing: [rocksdb/db/db_impl/db_impl_write.cc](https://github.com/facebook/rocksdb/blob/21d5a8f54f06e01ca49d4e6bae89bb42ba78dfdd/db/db_impl/db_impl_write.cc#L1706) Line 1706 in [21d5a8f](https://github.com/facebook/rocksdb/commit/21d5a8f54f06e01ca49d4e6bae89bb42ba78dfdd) status = SwitchMemtable(cfd, write_context); Having one check inside SwitchMemTable() will ensure there won't...

Moved the check to validate. This ensures accord flag is checked only during table creation and not during replay. patch by Pranav Shenoy; reviewed David by TBD for CASSANDRA-20600