>From Preetham Poluparthi <[email protected]>:

Attention is currently required from: Preetham Poluparthi.

Hello Anon. E. Moose #1000171, Jenkins,

I'd like you to reexamine a change. Please visit

    https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21169?usp=email

to look at the new patch set (#2).

The following approvals got outdated and were removed:
Contrib-2 by Anon. E. Moose #1000171, Integration-Tests+1 by Jenkins, 
Verified-1 by Jenkins


Change subject: [ASTERIXDB-3606][COMP]: Fix join condition handling for bushy 
plans
......................................................................

[ASTERIXDB-3606][COMP]: Fix join condition handling for bushy plans

 - user model changes: no
 - storage format changes: no
 - interface changes: no

Details:
Previously, join conditions were added assuming a zig zag join tree, which led 
to incorrect results for bushy plans. This change updates the logic to 
correctly handle join condition assignment in bushy join trees. This was seen 
when SelectPushDown was changed to bottom up in tpcds:q92
Ext-ref: MB-71042

Change-Id: I2c115afa0ab4e5b09046d75d8b344d39ce84e0fb
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinNode.java
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/tpcds/query-ASTERIXDB-1591.plan
2 files changed, 154 insertions(+), 164 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/69/21169/2
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21169?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings?usp=email

Gerrit-MessageType: newpatchset
Gerrit-Project: asterixdb
Gerrit-Branch: lumina
Gerrit-Change-Id: I2c115afa0ab4e5b09046d75d8b344d39ce84e0fb
Gerrit-Change-Number: 21169
Gerrit-PatchSet: 2
Gerrit-Owner: Preetham Poluparthi <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Attention: Preetham Poluparthi <[email protected]>

Reply via email to