Chen Li has posted comments on this change.

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


Patch Set 4:

@Wenhai: I want to take a closer look at this patch.  To help me and other 
reviewers, I wonder:

- Can you add high-level comments in the patch to explain the main changes you 
are making or the main bugs you are fixing?

- For those major changes, can we add brief comments to explain them?

- This patch looks big.  Is it possible to divide it to multiple, smaller 
patches?

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8736f104905eeda763d39709e002c2b9629278cc
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li <lwhaym...@yahoo.com>
Gerrit-Reviewer: Chen Li <che...@gmail.com>
Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Taewoo Kim <wangs...@yahoo.com>
Gerrit-Reviewer: Wenhai Li <lwhaym...@yahoo.com>
Gerrit-HasComments: No

Reply via email to