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

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/1350 to look at the new patch set (#32). 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-15 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 35: (1 comment)

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

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/1434 to look at the new patch set (#3). Change subject: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusExpressionToPlanTranslator

Change in asterixdb[master]: Add a corner case handling for NGramUTF8StringBinaryTokenizer

2017-01-17 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1448 Change subject: Add a corner case handling for NGramUTF8StringBinaryTokenizer .. Add a corner case handling for

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

2017-01-17 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 39: Could you also answer my previous question

Change in asterixdb[master]: Add a corner case handling for NGramUTF8StringBinaryTokenizer

2017-01-17 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1448 to look at the new patch set (#2). Change subject: Add a corner case handling for NGramUTF8StringBinaryTokenizer ..

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

2017-01-17 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 (#5). Change subject: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusExpressionToPlanTranslator

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

2017-01-17 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 39: (5 comments) A few comments on the commit

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

2017-01-17 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 39: (15 comments) A few more comments.

Change in asterixdb[master]: Add a corner case handling for NGramUTF8StringBinaryTokenizer

2017-01-17 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: Add a corner case handling for NGramUTF8StringBinaryTokenizer .. Add a corner case handling for NGramUTF8StringBinaryTokenizer - For a corner case where

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

2017-01-17 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 39: I have merged

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

2017-01-16 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 38: (7 comments) Sorry, a few more.

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

2017-01-16 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 38: (3 comments) Thanks for addressing the

Change in asterixdb-bad[master]: Fixed exceptions in BAD statements

2017-01-16 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Fixed exceptions in BAD statements .. Patch Set 1: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1443 To unsubscribe, visit

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

2017-01-16 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 (#4). Change subject: AQLPlus Refactoring 1: add a new rule controller .. AQLPlus

Change in asterixdb[master]: Ignore the license-automation-plugin on Eclipse

2017-01-16 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1446 Change subject: Ignore the license-automation-plugin on Eclipse .. Ignore the license-automation-plugin on Eclipse - Ignore a newly

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 (#8). 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 8: An unit test has not been added yet. I will let you know once I'm done. -- To view, visit

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

2017-01-17 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 (#6). 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-17 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 5: (8 comments)

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

2017-01-17 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 5: (1 comment)

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

2017-01-17 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 (#7). Change subject: AQLPlus Refactoring 1: add a new rule controller .. AQLPlus

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

2017-01-19 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: Please address my two comments in the patch set

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

2017-01-19 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 (#7). Change subject: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusExpressionToPlanTranslator

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

2017-01-19 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 (#6). 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-19 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 (#35). 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-19 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 (#34). 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-16 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 (#4). Change subject: AQLPlus Refactoring 2: AQL+ grammar generation and AQLPlusExpressionToPlanTranslator

Change in asterixdb[master]: Ignore the license-automation-plugin on Eclipse

2017-01-16 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: Ignore the license-automation-plugin on Eclipse .. Ignore the license-automation-plugin on Eclipse - Ignore a newly added license-automation-plugin on

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

2017-01-16 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 (#33). 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-16 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 38: (8 comments) A few more comments: 1. Could you

Change in asterixdb[master]: Fulltext search initial implementation

2016-08-18 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Fulltext search initial implementation .. Patch Set 14: 1) Is it really necessary to change every line of asterixdb/asterix-app/data/csv/fragile_02.adm? One of

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

2016-08-15 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 (#4). Change subject: ASTERIXDB-1566: Fixed External Hash Group By to conform to the memory budget

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

2016-08-15 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 (#5). Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple plan paths handling

Change in asterixdb[master]: ASTERIXDB-1186: fixed a composite primary index search issue.

2016-08-19 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new patch set (#2). Change subject: ASTERIXDB-1186: fixed a composite primary index search issue. .. ASTERIXDB-1186: fixed a composite primary index search issue. - Providing only one field to a

Change in asterixdb[master]: ASTERIXDB-1186: fixed a composite primary index search issue.

2016-08-19 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1094 Change subject: ASTERIXDB-1186: fixed a composite primary index search issue. .. ASTERIXDB-1186: fixed a composite primary index search

Change in asterixdb[master]: ASTERIXDB-1186: fixed a composite primary index search issue.

2016-08-19 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1094 to look at the new patch set (#3). Change subject: ASTERIXDB-1186: fixed a composite primary index search issue. ..

Change in asterixdb[master]: ASTERIXDB-1186: fixed a composite primary index search issue.

2016-08-20 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1094 to look at the new patch set (#4). Change subject: ASTERIXDB-1186: fixed a composite primary index search issue. ..

Change in asterixdb[master]: ASTERIXDB-1186: fixed a composite primary index search issue.

2016-08-20 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1094 to look at the new patch set (#5). Change subject: ASTERIXDB-1186: fixed a composite primary index search issue. ..

Change in asterixdb[master]: ASTERIXDB-1806: let inject_failure not utilize an index

2017-02-28 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1536 Change subject: ASTERIXDB-1806: let inject_failure not utilize an index .. ASTERIXDB-1806: let inject_failure not utilize an index -

Change in asterixdb[master]: ASTERIXDB-1806: let inject_failure not utilize an index

2017-02-28 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1536 to look at the new patch set (#4). Change subject: ASTERIXDB-1806: let inject_failure not utilize an index ..

Change in asterixdb[master]: ASTERIXDB-1806: let inject_failure not utilize an index

2017-02-28 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1536 to look at the new patch set (#3). Change subject: ASTERIXDB-1806: let inject_failure not utilize an index ..

Change in asterixdb[master]: ASTERIXDB-1806: let inject_failure not utilize an index

2017-02-28 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1806: let inject_failure not utilize an index .. Patch Set 1: (2 comments)

Change in asterixdb[master]: ASTERIXDB-1806: let inject_failure not utilize an index

2017-02-28 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1536 to look at the new patch set (#2). Change subject: ASTERIXDB-1806: let inject_failure not utilize an index ..

Change in asterixdb[master]: ASTERIXDB-1806: let inject_failure not utilize an index

2017-02-28 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1806: let inject_failure not utilize an index .. Patch Set 4: Added Steven since BAD test fails. -- To view, visit

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

2016-08-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 4: (8 comments) Some comments added.

Change in asterixdb[master]: ASTERIXDB-1566,ASTERIXDB-1628: Fixed External Hash Group By ...

2016-09-02 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1566,ASTERIXDB-1628: Fixed External Hash Group By to conform to the memory budget .. Patch Set 9: @Jianfeng: thanks. Will do. -- To view, visit

Change in asterixdb[master]: ASTERIXDB-1628: Fixed an issue in External Hash Group by

2016-09-02 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1144 Change subject: ASTERIXDB-1628: Fixed an issue in External Hash Group by .. ASTERIXDB-1628: Fixed an issue in External Hash Group by -

Change in asterixdb[master]: ASTERIXDB-1628: Fixed an issue in External Hash Group by

2016-09-03 Thread Taewoo Kim (Code Review)
Hello Jianfeng Jia, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1144 to look at the new patch set (#2). Change subject: ASTERIXDB-1628: Fixed an issue in External Hash Group by

Change in asterixdb[master]: ASTERIXDB-1628: Fixed an issue in External Hash Group by

2016-09-03 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1628: Fixed an issue in External Hash Group by .. Patch Set 2: Code-Review+2 Forwarding Jianfeng's +2 (just rebased) -- To view, visit

Change in asterixdb[master]: ASTERIXDB-1487: fix the wrong plan for inverted fuzzyjoin.

2016-09-09 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1487: fix the wrong plan for inverted fuzzyjoin. .. Patch Set 10: (1 comment)

Change in asterixdb[master]: Fulltext search initial implementation

2016-09-12 Thread Taewoo Kim (Code Review)
Hello Heri Ramampiaro, 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 (#15). Change subject: Fulltext search initial implementation ..

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

2016-09-14 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 (#12). 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-09-14 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 (#13). 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-09-14 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 (#11). Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple plan paths handling

Change in asterixdb[master]: Fulltext search initial implementation

2016-09-14 Thread Taewoo Kim (Code Review)
Hello Heri Ramampiaro, 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 (#17). Change subject: Fulltext search initial implementation ..

Change in asterixdb[master]: Adding a new 101 tutorial for SQL++.

2016-09-14 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Adding a new 101 tutorial for SQL++. .. Patch Set 1: (4 comments) Looks good to me!

Change in asterixdb[master]: Full-text search implementation step 1

2016-09-14 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1175 Change subject: Full-text search implementation step 1 .. Full-text search implementation step 1 - Changed a function name: contains

Change in asterixdb[master]: ASTERIXDB-1628: Fixed an issue in External Hash Group by

2016-09-08 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1628: Fixed an issue in External Hash Group by .. Patch Set 3: (1 comment)

Change in asterixdb[master]: ASTERIXDB-1628: Fixed an issue in External Hash Group by

2016-09-08 Thread Taewoo Kim (Code Review)
Hello Jianfeng Jia, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1144 to look at the new patch set (#3). Change subject: ASTERIXDB-1628: Fixed an issue in External Hash Group by

Change in asterixdb[master]: ASTERIXDB-1628: Fixed an issue in External Hash Group by

2016-09-08 Thread Taewoo Kim (Code Review)
Hello Jianfeng Jia, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1144 to look at the new patch set (#5). Change subject: ASTERIXDB-1628: Fixed an issue in External Hash Group by

Change in asterixdb[master]: ASTERIX-1487: fix the wrong plan for inverted fuzzyjoin.

2016-09-08 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIX-1487: fix the wrong plan for inverted fuzzyjoin. .. Patch Set 8: (3 comments)

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

2016-09-08 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 (#8). 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-09-08 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 (#7). Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple plan paths handling

Change in asterixdb[master]: ASTERIXDB-1566, ASTERIXDB-1628: Fixed External Hash Group By...

2016-09-09 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 (#10). Change subject: ASTERIXDB-1566, ASTERIXDB-1628: Fixed External Hash Group By to conform to the memory budget

Change in asterixdb[master]: ASTERIXDB-1628: Fixed an issue in External Hash Group by

2016-09-08 Thread Taewoo Kim (Code Review)
Hello Jianfeng Jia, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1144 to look at the new patch set (#6). Change subject: ASTERIXDB-1628: Fixed an issue in External Hash Group by

Change in asterixdb[master]: ASTERIXDB-1628: Fixed an issue in External Hash Group by

2016-09-09 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1628: Fixed an issue in External Hash Group by .. Patch Set 6: Code-Review+2 Forwarding Jianfeng's +2 (Rebased) -- To view, visit

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

2016-09-13 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 (#10). Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple plan paths handling

Change in asterixdb[master]: ASTERIXDB-1630: fix ATypeHierarchy to handle ANY.

2016-09-13 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1630: fix ATypeHierarchy to handle ANY. .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1168 To unsubscribe, visit

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

2016-09-12 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 10: (15 comments)

Change in asterixdb[master]: Full-text search implementation step 1

2016-09-15 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1175 to look at the new patch set (#2). Change subject: Full-text search implementation step 1 .. Full-text search

Change in asterixdb[master]: Full-text search implementation step 1

2016-09-15 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1175 to look at the new patch set (#3). Change subject: Full-text search implementation step 1 .. Full-text search

Change in asterixdb[master]: Full-text search implementation step 1

2016-09-15 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Full-text search implementation step 1 .. Patch Set 3: (1 comment)

Change in asterixdb[master]: ASTERIXDB-1566, ASTERIXDB-1628: Fixed External Hash Group By...

2016-09-15 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1566, ASTERIXDB-1628: Fixed External Hash Group By to conform to the memory budget .. Patch Set 10: (13 comments)

Change in asterixdb[master]: Full-text search implementation step 1

2016-09-15 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1175 to look at the new patch set (#7). Change subject: Full-text search implementation step 1 .. Full-text search

Change in asterixdb[master]: Updated DeletableFrameTupleAppender to support reusing index...

2016-09-26 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Updated DeletableFrameTupleAppender to support reusing index slots. .. Patch Set 1: (1 comment)

Change in asterixdb[master]: Tests for ASTERIXDB-1636

2016-10-05 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Tests for ASTERIXDB-1636 .. Patch Set 4: Code-Review+2 LGTM -- To view, visit https://asterix-gerrit.ics.uci.edu/1248 To unsubscribe, visit

Change in asterixdb[master]: ASTERIXDB-1671: fix non-positive limit with order by.

2016-10-04 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1671: fix non-positive limit with order by. .. Patch Set 4: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1240 To unsubscribe,

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

2016-10-04 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 (#17). Change subject: ASTERIXDB-1566: Fixed External Hash Group By to conform to the memory budget

Change in asterixdb[master]: Full-text implementation step 1

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

Change in asterixdb[master]: Index-only plan step 2: Added SplitOperator

2016-10-07 Thread Taewoo Kim (Code Review)
Hello Jianfeng Jia, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1196 to look at the new patch set (#14). Change subject: Index-only plan step 2: Added SplitOperator ..

Change in asterixdb[master]: Index-only plan step 2: Added SplitOperator

2016-10-04 Thread Taewoo Kim (Code Review)
Hello Jianfeng Jia, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1196 to look at the new patch set (#13). Change subject: Index-only plan step 2: Added SplitOperator ..

Change in asterixdb[master]: Tests for ASTERIXDB-1636

2016-10-05 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Tests for ASTERIXDB-1636 .. Patch Set 2: (3 comments)

Change in asterixdb[master]: Full-text search implementation step 1

2016-09-15 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1175 to look at the new patch set (#4). Change subject: Full-text search implementation step 1 .. Full-text search

Change in asterixdb[master]: Full-text search implementation step 1

2016-09-15 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1175 to look at the new patch set (#5). Change subject: Full-text search implementation step 1 .. Full-text search

Change in asterixdb[master]: Full-text search implementation step 1

2016-09-15 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Full-text search implementation step 1 .. Patch Set 6: @Yingyi: got it! Thanks. -- To view, visit https://asterix-gerrit.ics.uci.edu/1175 To unsubscribe, visit

Change in asterixdb[master]: Full-text search implementation step 1

2016-09-16 Thread Taewoo Kim (Code Review)
Taewoo Kim has abandoned this change. Change subject: Full-text search implementation step 1 .. Abandoned -- To view, visit https://asterix-gerrit.ics.uci.edu/1175 To unsubscribe, visit

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

2016-09-18 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 (#11). Change subject: ASTERIXDB-1566: Fixed External Hash Group By to conform to the memory budget

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

2016-09-18 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 (#15). Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple plan paths handling

Change in asterixdb[master]: ASTERIXDB-1566, ASTERIXDB-1628: Fixed External Hash Group By...

2016-09-16 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1566, ASTERIXDB-1628: Fixed External Hash Group By to conform to the memory budget .. Patch Set 9: (25 comments)

Change in asterixdb[master]: ASTERIXDB-1566, ASTERIXDB-1628: Fixed External Hash Group By...

2016-09-16 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1566, ASTERIXDB-1628: Fixed External Hash Group By to conform to the memory budget .. Patch Set 10: @Yingyi: thanks for the comments. -- To view,

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

2016-09-19 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 (#16). 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-09-19 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 (#17). Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple plan paths handling

Change in asterixdb[master]: Add Record To Map Methods

2016-09-22 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Add Record To Map Methods .. Patch Set 1: (1 comment)

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

2016-09-23 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 11: @Wenhai: can you answer to my comments in the

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

2016-09-23 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 (#14). Change subject: ASTERIXDB-1566: Fixed External Hash Group By to conform to the memory budget

Change in asterixdb[master]: Index-only plan step 2: Added SplitOperator

2016-09-23 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1196 to look at the new patch set (#4). Change subject: Index-only plan step 2: Added SplitOperator .. Index-only plan

Change in asterixdb[master]: Index-only plan step 2: Added SplitOperator

2016-09-21 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1196 to look at the new patch set (#2). Change subject: Index-only plan step 2: Added SplitOperator .. Index-only plan

Change in asterixdb[master]: Index-only plan step 2: Added SplitOperator

2016-09-22 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Index-only plan step 2: Added SplitOperator .. Patch Set 2: (4 comments)

Change in asterixdb[master]: Index-only plan step 2: Added SplitOperator

2016-09-22 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1196 to look at the new patch set (#3). Change subject: Index-only plan step 2: Added SplitOperator .. Index-only plan

<    1   2   3   4   5   6   7   8   >