rocksdb icon indicating copy to clipboard operation
rocksdb copied to clipboard

Should return Status::NotFound() when fullHistoryTsLow is empty

Open CARROTINA opened this issue 2 years ago • 1 comments

fixes #11093

Added a check to see if the fullHistoryTsLow is empty as in ts_low->size() == 0 and returned Status::NotFound()

CARROTINA avatar Jan 26 '23 08:01 CARROTINA

Not sure. Empty full_history_ts_low seems to unambiguously mean none is set. There might need to be a reason to change it. Then if we do change it, we'll need a release note in HISTORY.md.

The original reason in #11093 is no longer valid as the assertion changed to permit empty string

ajkr avatar Feb 20 '24 14:02 ajkr