refactor(relational_iter): dedup into_stream_inner and into_stream in StateTableRowIter

[wcy-fdu]

What's changed and what's your intention?

As title, using range of (Unbounded, Unbounded) for normal iter.

Checklist

• I have written necessary docs and comments
• I have added necessary unit tests and integration tests
• All checks passed in ./risedev check (or alias, ./risedev c)

Refer to a related PR or issue link (optional)

close #3049

[codecov]

Codecov Report

Merging #3057 (4aae399) into main (9af55da) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 4aae399 differs from pull request most recent head 132fa78. Consider uploading reports for the commit 132fa78 to get more accurate results

@@            Coverage Diff             @@
##             main    #3057      +/-   ##
==========================================
- Coverage   73.47%   73.46%   -0.01%     
==========================================
  Files         736      735       -1     
  Lines      100716   100477     -239     
==========================================
- Hits        73997    73816     -181     
+ Misses      26719    26661      -58     

Flag	Coverage Δ
rust	73.46% <100.00%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
src/storage/src/table/state_table.rs	86.79% <100.00%> (ø)
src/storage/src/table/cell_based_table.rs	69.91% <0.00%> (-2.70%)	⬇️
.../src/executor/managed_state/aggregation/extreme.rs	89.62% <0.00%> (-1.71%)	⬇️
src/storage/src/hummock/sstable/group_builder.rs	94.16% <0.00%> (-1.66%)	⬇️
src/meta/src/barrier/mod.rs	69.13% <0.00%> (-0.33%)	⬇️
src/meta/src/rpc/server.rs	0.00% <0.00%> (ø)
src/storage/src/keyspace.rs	87.62% <0.00%> (ø)
src/storage/src/hummock/mod.rs	85.52% <0.00%> (ø)
src/rpc_client/src/meta_client.rs	0.00% <0.00%> (ø)
src/meta/src/rpc/service/hummock_service.rs	0.00% <0.00%> (ø)
... and 5 more

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

[skyzh]

#3049 also raises an issue about bound.contains. We can do binary search to position the memtable iter instead of checking contains.

Redo

[wcy-fdu]

Directly pass BTreeMap::Range to iterating mem_table to avoid doing a comparison every time when we meet a memtable key.