[DRAFT] feat!: do not invalidate data block if bloomfilter returns false

[xxmplus]

do not invalidate data block if bloom filter returns false in SeekPrefixGE()

This is a DRAFT.

I don't get it why this proposal is beneficial 🤔. If I understand it correctly, the only valid operation after a successful SeekPrefixGE() is Next(). Now, if the prefix is definitely not in the block, Next() is then not a meaningful operation before another seek is called. What's the point to keep the data block around?

Also how do we prevent caller from calling Next() as mentioned in the comment? I must have missed something... 😕

It was necessary in earlier versions of the code since the caller was allowed to call Next when SeekPrefixGE returned nil. This is no longer allowed.

concerns: #4788

[cockroach-teamcity]

This change is