asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Taewoo Kim (Code Review)" <>
Subject Change in asterixdb[master]: Applied the multiway fuzzyjoin based on the prefix-based joi...
Date Tue, 17 Jan 2017 01:25:38 GMT
Taewoo Kim has posted comments on this change.

Change subject: Applied the multiway fuzzyjoin based on the prefix-based join and the selectFuzzyJoin

Patch Set 38:


A few more comments:

1. Could you tell me why the following test results are also affected?


2. Other than optimizer tests, have we added a few run-time tests as well?
File asterixdb/asterix-algebra/src/main/java/org/apache/asterix/compiler/provider/

Line 54:         defaultLogicalRewrites
This line should not be changed. Let's keep it as it is.
File asterixdb/asterix-app/src/test/resources/optimizerts/queries/fj-dblp-csx-hybrid.aql:

Line 20:  * Description    : Multiple fuzzy join on three tables, with a star join condition.
tables -> datasets
File asterixdb/asterix-app/src/test/resources/optimizerts/queries/fj-dblp-csx-selflink.aql:

Line 21:  *                  The base table DBLP will fuzzy join with CSX and propagate the
table -> dataset
File asterixdb/asterix-app/src/test/resources/optimizerts/queries/fj-dblp-csx-simple.aql:

Line 19: drop dataverse fj-dblp-csx if exists;
Please put a comment here - what this test does.

Line 26:   id: int32, 
Remove this white space.

Line 34:   id: int32, 
Remove this white space.
File asterixdb/asterix-app/src/test/resources/optimizerts/queries/fj-dblp-csx-star.aql:

Line 21:  *                  The CSX and DBLP tables are used twice and will be propagated
table -> dataset
File asterixdb/asterix-app/src/test/resources/runtimets/queries/fuzzyjoin/dblp-csx-4.1.1/word-jaccard.1.ddl.aql:

Line 21:  *                  We expect the join to be transformed into an prefix-based fuzzy
join following with a < select.
an -> a

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I8736f104905eeda763d39709e002c2b9629278cc
Gerrit-PatchSet: 38
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li <>
Gerrit-Reviewer: Chen Li <>
Gerrit-Reviewer: Jenkins <>
Gerrit-Reviewer: Taewoo Kim <>
Gerrit-Reviewer: Wenhai Li <>
Gerrit-HasComments: Yes

View raw message