-
Notifications
You must be signed in to change notification settings - Fork 526
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(optimizer): stream hop dist derive #3595
Conversation
Could you please enable |
@@ -52,3 +52,14 @@ | |||
StreamExchange { dist: HashShard([0]) } | |||
StreamProject { exprs: [($0 + $1), $0, $2] } | |||
StreamTableScan { table: t, columns: [a, b, _row_id], pk_indices: [2] } | |||
- sql: | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also provide logical_plan?
3fbd4ed
to
974e2e9
Compare
Signed-off-by: Bugen Zhao <i@bugenzhao.com>
Codecov Report
@@ Coverage Diff @@
## main #3595 +/- ##
==========================================
- Coverage 74.38% 74.38% -0.01%
==========================================
Files 771 771
Lines 109013 109016 +3
==========================================
+ Hits 81089 81090 +1
- Misses 27924 27926 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
Codecov Report
@@ Coverage Diff @@
## main #3595 +/- ##
=======================================
Coverage 74.37% 74.37%
=======================================
Files 771 771
Lines 109023 109026 +3
=======================================
+ Hits 81084 81091 +7
+ Misses 27939 27935 -4
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
* fix stream hop dist derive * give more phase for planner test * re open e2e * trigger ci Signed-off-by: Bugen Zhao <i@bugenzhao.com> Co-authored-by: Bugen Zhao <i@bugenzhao.com> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* fix stream hop dist derive * give more phase for planner test * re open e2e * trigger ci Signed-off-by: Bugen Zhao <i@bugenzhao.com> Co-authored-by: Bugen Zhao <i@bugenzhao.com> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
#3573