libroach: Don't assert on nonzero min timestamp hints in iterators #41217
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Currently, if either a min or max timestamp hint is provided to a
RocksDB iterator, we assert that both are non-zero. In practice,
we only need to assert on the max one being nonzero; a zero min
timestamp hint when encoded would sort before all timestamps, and
would not filter out any SSTs. A zero max timestamp
hint could filter out all SSTs, which would be worth asserting on.
Fixes #41200 .
Release justification: Fixes assertions that are currently failing
in tests, no risk otherwise.
Release note: None