Yingyi Bu has posted comments on this change. Change subject: Intersect the 2ndary indexes before primary search ......................................................................
Patch Set 2: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/577/2/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/physical/IntersectPOperator.java File algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/physical/IntersectPOperator.java: Line 89: List<ILocalStructuralProperty> propsLocal = op.getDeliveredPhysicalProperties().getLocalProperties(); > This logic here should be very similar to AbstractJoinPOperator.computeDeli What I meant was that the variables for describing those properties in this code is based on the variables live at "op", but those variables will not be live after "iop". For example, the first input branch is partitioned by $1 and ordered by $1, but after the intersect operator, $1 will be mapped to "iop".getOutputVars().get(0) and $1 will no longer be available in the output of "iop". -- To view, visit https://asterix-gerrit.ics.uci.edu/577 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic16c67c529ca19d8b1a5439ddef22760945fd0d7 Gerrit-PatchSet: 2 Gerrit-Project: hyracks Gerrit-Branch: master Gerrit-Owner: Jianfeng Jia <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Jianfeng Jia <[email protected]> Gerrit-Reviewer: Taewoo Kim <[email protected]> Gerrit-Reviewer: Yingyi Bu <[email protected]> Gerrit-Reviewer: Yingyi Bu <[email protected]> Gerrit-HasComments: Yes
