fix(streaming): lookup barrier is not aligned

[skyzh]

Signed-off-by: Alex Chi iskyzh@gmail.com

What's changed and what's your intention?

Lookup should forward barrier only when two sides are both ready.

Checklist

• I have written necessary docs and comments
• I have added necessary unit tests and integration tests

Refer to a related PR or issue link (optional)

close #2243

[codecov]

Codecov Report

Merging #2244 (d5ce477) into main (5c3a32e) will increase coverage by 0.06%.
The diff coverage is 40.00%.

@@            Coverage Diff             @@
##             main    #2244      +/-   ##
==========================================
+ Coverage   71.03%   71.09%   +0.06%     
==========================================
  Files         657      657              
  Lines       83592    83865     +273     
==========================================
+ Hits        59378    59624     +246     
- Misses      24214    24241      +27     

Flag	Coverage Δ
rust	71.09% <40.00%> (+0.06%)	⬆️

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

Impacted Files	Coverage Δ
src/stream/src/executor_v2/lookup/impl_.rs	95.08% <ø> (ø)
src/stream/src/executor_v2/lookup/sides.rs	29.16% <ø> (ø)
src/stream/src/executor/dispatch.rs	76.81% <37.50%> (-0.37%)	⬇️
src/stream/src/executor/actor.rs	87.09% <50.00%> (-6.24%)	⬇️
...rc/hummock/shared_buffer/shared_buffer_uploader.rs	81.73% <0.00%> (-5.77%)	⬇️
src/storage/src/hummock/shared_buffer/mod.rs	90.18% <0.00%> (-4.56%)	⬇️
src/frontend/src/expr/subquery.rs	46.42% <0.00%> (-1.85%)	⬇️
...e/src/hummock/shared_buffer/shared_buffer_batch.rs	94.16% <0.00%> (-1.47%)	⬇️
src/storage/src/hummock/mod.rs	85.54% <0.00%> (-1.37%)	⬇️
... and 17 more

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