refactor: refactor create state table code + fix incorrect arguments

[BowenXiao1999]

What's changed and what's your intention?

It was bug introduced by incorrect arguments (My bad)...

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)

[codecov]

Codecov Report

Merging #3190 (edf888f) into main (6d98333) will decrease coverage by 0.00%.
The diff coverage is 97.36%.

@@            Coverage Diff             @@
##             main    #3190      +/-   ##
==========================================
- Coverage   73.57%   73.56%   -0.01%     
==========================================
  Files         744      744              
  Lines      102180   102202      +22     
==========================================
+ Hits        75174    75182       +8     
- Misses      27006    27020      +14     

Flag	Coverage Δ
rust	73.56% <97.36%> (-0.01%)	⬇️

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

Impacted Files	Coverage Δ
src/stream/src/executor/aggregation/mod.rs	92.55% <95.83%> (-2.41%)	⬇️
src/stream/src/executor/global_simple_agg.rs	96.06% <100.00%> (-0.02%)	⬇️
src/stream/src/executor/hash_agg.rs	97.61% <100.00%> (-0.01%)	⬇️
src/meta/src/hummock/mock_hummock_meta_client.rs	42.39% <0.00%> (-1.09%)	⬇️
src/storage/src/table/cell_based_table.rs	69.77% <0.00%> (-0.82%)	⬇️
src/common/src/types/ordered_float.rs	24.70% <0.00%> (-0.20%)	⬇️
src/storage/src/hummock/local_version_manager.rs	84.12% <0.00%> (-0.16%)	⬇️
src/meta/src/barrier/mod.rs	69.45% <0.00%> (+0.32%)	⬆️

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

[BugenZhao]

LGTM. BTW, which bug does this PR fix?

[skyzh]

Would you please help check how we encode stream pk previously? For max agg, did we also encode pk in descending way?

[BowenXiao1999]

You mean write batch/keyspace? I think after this PR, as long as we are using state table, we are encode pk in correct order way.