You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Now we don't have a way to force backend executor construct its ExecutorInfo with the pk indices derived in optimizer. And thus many executors derive pk indices again in their new with input.pk_indices(). This can lead to bugs like #12282 (fixed by #12304) and #10879 (fixed by #11276).
See #12763, enabling the assertion suggested by @BugenZhao will make e2e failed. We may need to revisit every executor to ensure the correctness.
Now we don't have a way to force backend executor construct its
ExecutorInfo
with the pk indices derived in optimizer. And thus many executors derive pk indices again in theirnew
withinput.pk_indices()
. This can lead to bugs like #12282 (fixed by #12304) and #10879 (fixed by #11276).See #12763, enabling the assertion suggested by @BugenZhao will make e2e failed. We may need to revisit every executor to ensure the correctness.
Failed e2e and and the panicked executors:
e2e_test/streaming/aggregate/hdr_approx_percentile.slt
:TopN
e2e_test/streaming/append_only.slt
:Materialize
e2e_test/streaming/bug_fixes/hop_window_update_10495.slt
:Materialize
e2e_test/streaming/bug_fixes/issue_12299.slt
:Materialize
e2e_test/streaming/bug_fixes/issue_5285.slt
:TopN
e2e_test/streaming/bug_fixes/issue_5825.slt
:TopN
e2e_test/streaming/bug_fixes/issue_8084.slt
:Materialize
e2e_test/streaming/bug_fixes/issue_8570.slt
:TopN
e2e_test/streaming/bug_fixes/mv_on_singleton_mv_4153.slt
:TopN
e2e_test/streaming/delta_join/delta_join_snapshot.slt
:Materialize
e2e_test/streaming/delta_join/delta_join_snapshot_no_index.slt
:Materialize
e2e_test/streaming/delta_join/delta_join_upstream.slt
:Materialize
e2e_test/streaming/delta_join/delta_join_upstream_no_index.slt
:Materialize
e2e_test/streaming/group_top_n/main1.slt
:TopN
e2e_test/streaming/index.slt
:Materialize
e2e_test/streaming/natural_and_cross_join.slt
:Materialize
e2e_test/streaming/nexmark_snapshot.slt
:TopN
e2e_test/streaming/nexmark_upstream.slt
:TopN
e2e_test/streaming/order_by.slt
:TopN
e2e_test/streaming/outer_join.slt
:Materialize
e2e_test/streaming/struct_table.slt
:Materialize
e2e_test/streaming/temporal_join/temporal_join.slt
:Materialize
e2e_test/streaming/temporal_join/temporal_join_watermark.slt
:Materialize
e2e_test/streaming/temporal_join/temporal_join_with_index.slt
:Materialize
e2e_test/streaming/tpch_snapshot.slt
:TopN
e2e_test/streaming/tpch_upstream.slt
:TopN
The text was updated successfully, but these errors were encountered: