Wenhai Li has posted comments on this change.

Change subject: ASTERIX-1487: fix the wrong plan for inverted fuzzyjoin.
......................................................................


Patch Set 8:

@Chen
It's a quite strange query, the high-level comments towards the example is like:
1. We setup a fuzzy join over A.a ~= B.b based on a word index over A.a
2. We want to aggregate on B's fields.
If we switch the consequence of A and B in the two "FOR" lines, we get 
inconsistent result. The RemoveUnusedOneToOneEuquiJoinRule remove the branch of 
A after "pseudo" broadcase B to A for enabling "SELECT" instead of fuzzy join. 
But that is wrong.

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1119
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1aef69a2278853fd9f8020da6639331b367ed5ad
Gerrit-PatchSet: 8
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li <[email protected]>
Gerrit-Reviewer: Chen Li <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Jianfeng Jia <[email protected]>
Gerrit-Reviewer: Taewoo Kim <[email protected]>
Gerrit-Reviewer: Till Westmann <[email protected]>
Gerrit-Reviewer: Wenhai Li <[email protected]>
Gerrit-Reviewer: Yingyi Bu <[email protected]>
Gerrit-Reviewer: Yingyi Bu <[email protected]>
Gerrit-HasComments: No

Reply via email to