Change in asterixdb[master]: Index-only plan

2016-06-21 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Index-only plan .. Patch Set 9: (238 comments)

Change in asterixdb[master]: ASTERIXDB-1544: fix the issue with fuzzyjoin on an indexed n...

2016-07-26 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1544: fix the issue with fuzzyjoin on an indexed nullable field .. Patch Set 2: Thanks Yingyi and Wenhai. -- To view, visit

Change in asterixdb[master]: ASTERIXDB-1544: fix the issue with fuzzyjoin on an indexed n...

2016-07-25 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1544: fix the issue with fuzzyjoin on an indexed nullable field .. Patch Set 1: @Chen: I have uploaded this patch. Wenhai will review this. :-) -- To

Change in asterixdb[master]: ASTERIXDB-1544: fix the issue with fuzzyjoin on an indexed n...

2016-07-25 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1544: fix the issue with fuzzyjoin on an indexed nullable field .. Patch Set 1: @Chen: this is a trivial patch. -- To view, visit

Change in asterixdb[master]: ASTERIXDB-1544: fix the issue with fuzzyjoin on an indexed n...

2016-07-25 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1021 Change subject: ASTERIXDB-1544: fix the issue with fuzzyjoin on an indexed nullable field .. ASTERIXDB-1544: fix the issue with

Change in asterixdb[master]: Fulltext search initial implementation

2016-07-22 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/989 to look at the new patch set (#7). Change subject: Fulltext search initial implementation .. Fulltext search initial

Change in asterixdb[master]: ASTERIXDB-1154: index-left-outer-join with LIMIT now works.

2016-07-22 Thread Taewoo Kim (Code Review)
Taewoo Kim has abandoned this change. Change subject: ASTERIXDB-1154: index-left-outer-join with LIMIT now works. .. Abandoned -- To view, visit https://asterix-gerrit.ics.uci.edu/470 To unsubscribe, visit

Change in asterixdb[master]: Fulltext search initial implementation

2016-07-22 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/989 to look at the new patch set (#6). Change subject: Fulltext search initial implementation .. Fulltext search initial

Change in incubator-asterixdb-site[asf-site]: community and dev-setup fixes

2016-07-21 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: community and dev-setup fixes .. Patch Set 3: (1 comment) Since we are dealing with two IDEs, it may be nice to point both.

Change in asterixdb[master]: Fulltext search initial implementation

2016-07-22 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/989 to look at the new patch set (#8). Change subject: Fulltext search initial implementation .. Fulltext search initial

Change in asterixdb[master]: Fulltext search initial implementation

2016-07-22 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/989 to look at the new patch set (#9). Change subject: Fulltext search initial implementation .. Fulltext search initial

Change in asterixdb[master]: Fulltext search initial implementation

2016-07-21 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Fulltext search initial implementation .. Patch Set 5: @Mike: most of the files that you put the comments are not the file that I have changed. Do you know what

Change in asterixdb[master]: Fulltext search initial implementation

2016-07-21 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Fulltext search initial implementation .. Patch Set 5: (1 comment) Sent you an e-mail.

Change in asterixdb[master]: ASTERIXDB-1569: the config file now supports unit, comma, an...

2016-08-10 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1064 Change subject: ASTERIXDB-1569: the config file now supports unit, comma, and decimal. .. ASTERIXDB-1569: the config file now supports

Change in asterixdb[master]: ASTERIXDB-1569: the config file now supports unit, comma, an...

2016-08-11 Thread Taewoo Kim (Code Review)
Hello Yingyi Bu, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1064 to look at the new patch set (#3). Change subject: ASTERIXDB-1569: the config file now supports unit, comma, and decimal.

Change in asterixdb[master]: ASTERIXDB-1569: the config file now supports unit, comma, an...

2016-08-11 Thread Taewoo Kim (Code Review)
Hello Yingyi Bu, Till Westmann, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1064 to look at the new patch set (#4). Change subject: ASTERIXDB-1569: the config file now supports unit, comma, and decimal.

Change in asterixdb[master]: ASTERIXDB-1569: the config file now supports unit, comma, an...

2016-08-12 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1569: the config file now supports unit, comma, and decimal. .. Patch Set 1: (3 comments)

Change in asterixdb[master]: ASTERIXDB-1569: the config file now supports unit, comma, an...

2016-08-12 Thread Taewoo Kim (Code Review)
Hello Yingyi Bu, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1064 to look at the new patch set (#2). Change subject: ASTERIXDB-1569: the config file now supports unit, comma, and decimal.

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multi...

2016-08-12 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1073 to look at the new patch set (#3). Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple plan paths handling

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multi...

2016-08-12 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1073 to look at the new patch set (#4). Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple plan paths handling

Change in asterixdb[master]: Index-only plan

2016-08-12 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Index-only plan .. Patch Set 23: (3 comments) Thanks Till. I will breakdown this big change into smaller pieces as much as possible and upload them as a new

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple pat...

2016-08-12 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple paths in a plan can be handled. .. Patch Set 2: (6 comments)

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple pat...

2016-08-12 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1073 Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple paths in a plan can be handled. .. ASTERIXDB-1081,

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple pat...

2016-08-12 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple paths in a plan can be handled. .. Patch Set 1: This patch set is uploaded as a subset of Index-only

Change in asterixdb[master]: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple pat...

2016-08-12 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1073 to look at the new patch set (#2). Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: multiple paths in a plan can be handled.

Change in asterixdb[master]: Fulltext search initial implementation

2016-07-12 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/989 to look at the new patch set (#3). Change subject: Fulltext search initial implementation .. Fulltext search initial

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

2016-07-20 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Update dev-setup to use IntelliJ instead .. Patch Set 2: (2 comments) It looks good. I agree with Till regarding his comments.

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

2016-07-20 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Update dev-setup to use IntelliJ instead .. Patch Set 6: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1005 To unsubscribe, visit

Change in incubator-asterixdb-site[asf-site]: Update dev-setup to use IntelliJ instead

2016-07-20 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Update dev-setup to use IntelliJ instead .. Patch Set 5: (2 comments) two whitespaces https://asterix-gerrit.ics.uci.edu/#/c/1005/5/dev-setup.md File

Change in asterixdb[master]: Index-only plan

2016-07-17 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Index-only plan .. Patch Set 14: Regarding Young-Seok's comments: 1. R-tree index on rectangle type is not correctly dealt with for the corresponding index-only

Change in asterixdb[master]: ASTERIXDB-1538: fix the issue where LSMRTree index instance ...

2016-07-19 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1010 Change subject: ASTERIXDB-1538: fix the issue where LSMRTree index instance is created during the recovery process. .. ASTERIXDB-1538:

Change in asterixdb[master]: ASTERIXDB-1538: fix an issue where LSMRTree index instance i...

2016-07-20 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1010 to look at the new patch set (#4). Change subject: ASTERIXDB-1538: fix an issue where LSMRTree index instance is created during the recovery process.

Change in asterixdb[master]: ASTERIXDB-1538: fix an issue where LSMRTree index instance i...

2016-07-20 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1010 to look at the new patch set (#3). Change subject: ASTERIXDB-1538: fix an issue where LSMRTree index instance is created during the recovery process.

Change in asterixdb[master]: ASTERIXDB-1538: fix the issue where LSMRTree index instance ...

2016-07-20 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1010 to look at the new patch set (#2). Change subject: ASTERIXDB-1538: fix the issue where LSMRTree index instance is created during the recovery process.

Change in asterixdb[master]: Fulltext search initial implementation

2016-07-15 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Fulltext search initial implementation .. Patch Set 4: (4 comments) Thanks for the comments.

Change in asterixdb[master]: Fulltext search initial implementation - Supports Fulltext ...

2016-07-11 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/989 Change subject: Fulltext search initial implementation - Supports Fulltext search using XQuery syntax - Keyword level AND or OR search is possible - Changed a function name: contains -> string-contains

Change in asterixdb[master]: Fulltext search initial implementation - Supports Fulltext ...

2016-07-11 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/989 to look at the new patch set (#2). Change subject: Fulltext search initial implementation - Supports Fulltext search using XQuery syntax - Keyword level AND or OR search is possible

Change in asterixdb[master]: Index-only plan

2016-07-16 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Index-only plan .. Patch Set 11: (18 comments)

Change in asterixdb[master]: Index-only plan

2016-07-15 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Index-only plan .. Patch Set 11: Addressed Till's comments. -- To view, visit https://asterix-gerrit.ics.uci.edu/744 To unsubscribe, visit

Change in incubator-asterixdb-site[asf-site]: Update the dev-setup document to reflect recent changes - r...

2016-07-08 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: Update the dev-setup document to reflect recent changes - removed incubator string - revised instruction to import asterixdb only .. Update the

Change in incubator-asterixdb-site[asf-site]: Update the dev-setup document to reflect recent changes - r...

2016-07-08 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/983 Change subject: Update the dev-setup document to reflect recent changes - removed incubator string - revised instruction to import asterixdb only

Change in asterixdb[master]: ASTERIXDB-1566: Fixed External Hash Group By to conform to t...

2016-08-08 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1056 Change subject: ASTERIXDB-1566: Fixed External Hash Group By to conform to the memory budget .. ASTERIXDB-1566: Fixed External Hash

Change in asterixdb[master]: ASTERIXDB-1566: Fixed External Hash Group By to conform to t...

2016-08-08 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1056 to look at the new patch set (#2). Change subject: ASTERIXDB-1566: Fixed External Hash Group By to conform to the memory budget

Change in asterixdb[master]: Fulltext search initial implementation

2016-07-23 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Fulltext search initial implementation .. Patch Set 5: (1 comment)

Change in asterixdb[master]: Fulltext search initial implementation

2016-07-23 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/989 to look at the new patch set (#10). Change subject: Fulltext search initial implementation .. Fulltext search initial

Change in asterixdb[master]: ASTERIXDB-1459: Enable Debugging for Generated Functions

2016-07-29 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1459: Enable Debugging for Generated Functions .. Patch Set 2: Tried and it works fine on Eclipse. I tried setting a breakpoint in

Change in asterixdb[master]: Fulltext search initial implementation

2016-07-28 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/989 to look at the new patch set (#11). Change subject: Fulltext search initial implementation .. Fulltext search initial

Change in asterixdb[master]: Attempt To Fix Checkstyle Plugin In Eclipse

2016-06-30 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Attempt To Fix Checkstyle Plugin In Eclipse .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/968 To unsubscribe, visit

Change in asterixdb[master]: ASTERIXDB-1566: Fixed External Hash Group By to conform to t...

2016-08-16 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1056 to look at the new patch set (#7). Change subject: ASTERIXDB-1566: Fixed External Hash Group By to conform to the memory budget

Change in asterixdb[master]: ASTERIXDB-1566: Fixed External Hash Group By to conform to t...

2016-08-16 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1566: Fixed External Hash Group By to conform to the memory budget .. Patch Set 5: (14 comments) Thanks Jianfeng!

Change in asterixdb[master]: ASTERIXDB-1566: Fixed External Hash Group By to conform to t...

2016-08-17 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1056 to look at the new patch set (#8). Change subject: ASTERIXDB-1566: Fixed External Hash Group By to conform to the memory budget

Change in asterixdb[master]: Index-only plan step 3: Top-down Select and Join transformat...

2017-02-02 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1350 to look at the new patch set (#42). Change subject: Index-only plan step 3: Top-down Select and Join transformation rule

Change in asterixdb[master]: ASTERIXDB-1778: optimize the edit-distance-check function

2017-02-02 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1481 Change subject: ASTERIXDB-1778: optimize the edit-distance-check function .. ASTERIXDB-1778: optimize the edit-distance-check function

Change in asterixdb[master]: ASTERIXDB-1778: Optimize the edit-distance-check function

2017-02-03 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1481 to look at the new patch set (#5). Change subject: ASTERIXDB-1778: Optimize the edit-distance-check function ..

Change in asterixdb[master]: ASTERIXDB-1778: Optimize the edit-distance-check function

2017-02-03 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1778: Optimize the edit-distance-check function .. Patch Set 5: @Jianfeng: the early termination logic is in place. We have test cases for them, too. In

Change in asterixdb[master]: ASTERIXDB-1778: Optimize the edit-distance-check function

2017-02-03 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1778: Optimize the edit-distance-check function .. Patch Set 3: (11 comments) @Jianfeng: Thanks!

Change in asterixdb[master]: ASTERIXDB-1778: optimize the edit-distance-check function

2017-02-02 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1778: optimize the edit-distance-check function .. Patch Set 2: Added Steven because of BAD failure. -- To view, visit

Change in asterixdb[master]: ASTERIXDB-1778: Optimize the edit-distance-check function

2017-02-04 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1481 to look at the new patch set (#7). Change subject: ASTERIXDB-1778: Optimize the edit-distance-check function ..

Change in asterixdb[master]: ASTERIXDB-1778: Optimize the edit-distance-check function

2017-02-05 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: ASTERIXDB-1778: Optimize the edit-distance-check function .. ASTERIXDB-1778: Optimize the edit-distance-check function - Only calculate 2 * (threshold +

Change in asterixdb[master]: ASTERIXDB-1778: Optimize the edit-distance-check function

2017-02-04 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1481 to look at the new patch set (#8). Change subject: ASTERIXDB-1778: Optimize the edit-distance-check function ..

Change in asterixdb[master]: ASTERIXDB-1778: optimize the edit-distance-check function

2017-02-02 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1481 to look at the new patch set (#3). Change subject: ASTERIXDB-1778: optimize the edit-distance-check function ..

Change in asterixdb[master]: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusEx...

2017-02-02 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1434 to look at the new patch set (#14). Change subject: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusExpressionToPlanTranslator

Change in asterixdb[master]: Index-only plan step 3: Top-down Select and Join transformat...

2017-02-02 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1350 to look at the new patch set (#43). Change subject: Index-only plan step 3: Top-down Select and Join transformation rule

Change in asterixdb[master]: Index-only plan step 3: Top-down Select and Join transformat...

2017-02-02 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Index-only plan step 3: Top-down Select and Join transformation rule .. Patch Set 42: Added Steven because of BAD failure. -- To view, visit

Change in asterixdb[master]: ASTERIXDB-1778: optimize the edit-distance-check function

2017-02-02 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1481 to look at the new patch set (#2). Change subject: ASTERIXDB-1778: optimize the edit-distance-check function ..

Change in asterixdb[master]: ASTERIXDB-1778: Optimize the edit-distance-check function

2017-02-02 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1481 to look at the new patch set (#4). Change subject: ASTERIXDB-1778: Optimize the edit-distance-check function ..

Change in asterixdb[master]: ASTERIXDB-1778: Optimize the edit-distance-check function

2017-02-03 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1778: Optimize the edit-distance-check function .. Patch Set 6: For your comments, edit-distance-check_strings test case already contains that corner

Change in asterixdb[master]: ASTERIXDB-1778: Optimize the edit-distance-check function

2017-02-04 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1778: Optimize the edit-distance-check function .. Patch Set 6: @Jianfeng: I now see what you mean. Since the main function is a private function, yes,

Change in asterixdb[master]: Applied the multiway fuzzyjoin based on the prefix-based joi...

2017-01-22 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Applied the multiway fuzzyjoin based on the prefix-based join and the selectFuzzyJoin testCases. .. Patch Set 47: (10 comments) A few more comments. Besides,

Change in asterixdb[master]: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusEx...

2017-01-21 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1434 to look at the new patch set (#8). Change subject: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusExpressionToPlanTranslator

Change in asterixdb[master]: Index-only plan step 3: Top-down Select and Join transformat...

2017-01-21 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1350 to look at the new patch set (#36). Change subject: Index-only plan step 3: Top-down Select and Join transformation rule

Change in asterixdb[master]: Index-only plan step 3: Top-down Select and Join transformat...

2017-01-24 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1350 to look at the new patch set (#37). Change subject: Index-only plan step 3: Top-down Select and Join transformation rule

Change in asterixdb[master]: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusEx...

2017-01-24 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1434 to look at the new patch set (#9). Change subject: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusExpressionToPlanTranslator

Change in asterixdb[master]: Index-only plan step 3: Top-down Select and Join transformat...

2017-01-26 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1350 to look at the new patch set (#39). Change subject: Index-only plan step 3: Top-down Select and Join transformation rule

Change in asterixdb[master]: Index-only plan step 3: Top-down Select and Join transformat...

2017-01-30 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1350 to look at the new patch set (#40). Change subject: Index-only plan step 3: Top-down Select and Join transformation rule

Change in asterixdb[master]: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusEx...

2017-01-30 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1434 to look at the new patch set (#12). Change subject: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusExpressionToPlanTranslator

Change in asterixdb[master]: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusEx...

2017-01-25 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1434 to look at the new patch set (#10). Change subject: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusExpressionToPlanTranslator

Change in asterixdb[master]: Index-only plan step 3: Top-down Select and Join transformat...

2017-01-25 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1350 to look at the new patch set (#38). Change subject: Index-only plan step 3: Top-down Select and Join transformation rule

Change in asterixdb[master]: Applied the multiway fuzzyjoin based on the prefix-based joi...

2017-01-20 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Applied the multiway fuzzyjoin based on the prefix-based join and the selectFuzzyJoin testCases. .. Patch Set 45: (8 comments) Still more comments...

Change in asterixdb[master]: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusEx...

2017-01-26 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1434 to look at the new patch set (#11). Change subject: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusExpressionToPlanTranslator

Change in asterixdb[master]: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusEx...

2017-02-16 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusExpressionToPlanTranslator .. AQLPlus Refactoring 2: AQL+ grammar generation and

Change in asterixdb[master]: Truncate a temporary file before deleting it.

2017-02-11 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Truncate a temporary file before deleting it. .. Patch Set 3: (1 comment) Thanks Yingyi.

Change in asterixdb[master]: Truncate a temporary file before deleting it.

2017-02-11 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1501 to look at the new patch set (#4). Change subject: Truncate a temporary file before deleting it. .. Truncate a

Change in asterixdb[master]: ASTERIXDB-1791: Add a sanity check to hashjoin partition rea...

2017-02-13 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1505 to look at the new patch set (#2). Change subject: ASTERIXDB-1791: Add a sanity check to hashjoin partition reading

Change in asterixdb[master]: Truncate a temporary file before deleting it.

2017-02-10 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1501 Change subject: Truncate a temporary file before deleting it. .. Truncate a temporary file before deleting it. - Before RunFileReader

Change in asterixdb[master]: Truncate a temporary file before deleting it.

2017-02-10 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1501 to look at the new patch set (#2). Change subject: Truncate a temporary file before deleting it. .. Truncate a

Change in asterixdb[master]: Truncate a temporary file before deleting it.

2017-02-10 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1501 to look at the new patch set (#3). Change subject: Truncate a temporary file before deleting it. .. Truncate a

Change in asterixdb[master]: Truncate a temporary file before deleting it.

2017-02-12 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1501 to look at the new patch set (#5). Change subject: Truncate a temporary file before deleting it. .. Truncate a

Change in asterixdb[master]: Truncate a temporary file before deleting it.

2017-02-12 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Truncate a temporary file before deleting it. .. Patch Set 4: (1 comment)

Change in asterixdb[master]: ASTERIXDB-1791: Add a sanity check to hashjoin partition rea...

2017-02-13 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1505 Change subject: ASTERIXDB-1791: Add a sanity check to hashjoin partition reading .. ASTERIXDB-1791: Add a sanity check to hashjoin

Change in asterixdb[master]: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusEx...

2017-02-15 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusExpressionToPlanTranslator .. Patch Set 17: (1 comment)

Change in asterixdb[master]: AQLPlus Refactoring 1: add a new rule controller

2017-01-18 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1432 to look at the new patch set (#9). Change subject: AQLPlus Refactoring 1: add a new rule controller .. AQLPlus

Change in asterixdb[master]: AQLPlus Refactoring 1: add a new rule controller

2017-01-18 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: AQLPlus Refactoring 1: add a new rule controller .. Patch Set 11: (2 comments)

Change in asterixdb[master]: AQLPlus Refactoring 1: add a new rule controller

2017-01-18 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1432 to look at the new patch set (#10). Change subject: AQLPlus Refactoring 1: add a new rule controller .. AQLPlus

Change in asterixdb[master]: AQLPlus Refactoring 1: add a new rule controller

2017-01-18 Thread Taewoo Kim (Code Review)
Hello Yingyi Bu, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1432 to look at the new patch set (#12). Change subject: AQLPlus Refactoring 1: add a new rule controller ..

Change in asterixdb[master]: AQLPlus Refactoring 1: add a new rule controller

2017-01-18 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: AQLPlus Refactoring 1: add a new rule controller .. AQLPlus Refactoring 1: add a new rule controller - Add a new rule controller that is to be used for

Change in asterixdb[master]: Applied the multiway fuzzyjoin based on the prefix-based joi...

2017-01-18 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Applied the multiway fuzzyjoin based on the prefix-based join and the selectFuzzyJoin testCases. .. Patch Set 44: A new change regarding the optimization

Change in asterixdb[master]: Applied the multiway fuzzyjoin based on the prefix-based joi...

2017-01-18 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Applied the multiway fuzzyjoin based on the prefix-based join and the selectFuzzyJoin testCases. .. Patch Set 43: (2 comments)

Change in asterixdb[master]: AQLPlus Refactoring 1: add a new rule controller

2017-01-15 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1432 to look at the new patch set (#3). Change subject: AQLPlus Refactoring 1: add a new rule controller .. AQLPlus

Change in asterixdb[master]: Change the compilation related Exception

2017-01-15 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1437 to look at the new patch set (#8). Change subject: Change the compilation related Exception .. Change the

  1   2   3   4   5   6   7   8   >