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

2016-09-20 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple plan paths handling .. ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple

Change in asterixdb[master]: ASTERIXDB-1487: fix the wrong plan when we purning the selec...

2016-09-20 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1487: fix the wrong plan when we purning the selective branch. .. Patch Set 12: I discussed this with Wenhai. For a join such as A.x = B.y, we need to

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

2016-09-20 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple plan paths handling .. Patch Set 20: Code-Review+2 Forwarding Yingyi's +2 (just rebased) -- To

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

2016-09-20 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 (#13). 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-20 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/1073 to look at the new patch set (#20). 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-16 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 (#14). Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple plan paths handling

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

2016-09-19 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 11: (2 comments)

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

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

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/1056 to look at the new patch set (#12). 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-19 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multiple plan paths handling .. Patch Set 17: (5 comments)

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

Change in asterixdb[master]: Fulltext search initial implementation

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/989 to look at the new patch set (#14). Change subject: Fulltext search initial implementation .. Fulltext search initial

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

2016-08-25 Thread Taewoo Kim (Code Review)
Hello Michael Carey, Yingyi Bu, 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 (#7). Change subject: ASTERIXDB-1186: fixed a composite primary index search issue.

Change in asterixdb[master]: ASTERIXDB-1088: fixed a secondary index compact issue

2016-08-27 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1123 Change subject: ASTERIXDB-1088: fixed a secondary index compact issue .. ASTERIXDB-1088: fixed a secondary index compact issue -

Change in asterixdb[master]: ASTERIXDB-1088: fixed a secondary index compact issue

2016-08-28 Thread Taewoo Kim (Code Review)
Hello Young-Seok Kim, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1123 to look at the new patch set (#2). Change subject: ASTERIXDB-1088: fixed a secondary index compact issue

Change in asterixdb[master]: ASTERIXDB-1088: fixed a secondary index compact issue

2016-08-28 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: ASTERIXDB-1088: fixed a secondary index compact issue .. ASTERIXDB-1088: fixed a secondary index compact issue - Secondary index is now properly

Change in asterixdb[master]: ASTERIXDB-1088: fixed a secondary index compact issue

2016-08-28 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1088: fixed a secondary index compact issue .. Patch Set 2: Forwarding Young-Seok's +2 (just rebased) -- To view, visit

Change in asterixdb[master]: ASTERIXDB-1088: fixed a secondary index compact issue

2016-08-28 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1088: fixed a secondary index compact issue .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1123 To unsubscribe,

Change in asterixdb[master]: Fix Numeric Overflow For Units > GB

2016-08-28 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Fix Numeric Overflow For Units > GB .. Patch Set 3: Code-Review+2 LGTM -- To view, visit https://asterix-gerrit.ics.uci.edu/1084 To unsubscribe, visit

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

2016-08-25 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1186: fixed a composite primary index search issue. .. Patch Set 7: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1094 To

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

2016-08-25 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1186: fixed a composite primary index search issue. .. Patch Set 7: Forwarding Yingyi's +2 from the patch set 6 (just rebased) -- To view, visit

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

2016-08-25 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: ASTERIXDB-1186: fixed a composite primary index search issue. .. ASTERIXDB-1186: fixed a composite primary index search issue. - Providing only one field

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

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

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

2016-09-28 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 (#9). Change subject: Index-only plan step 2: Added SplitOperator .. Index-only plan

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

2016-09-28 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Updated DeletableFrameTupleAppender to support reusing index slots. .. Patch Set 2: Before going into the detail, I would like to add my two-cent thoughts since

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

2016-09-28 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 (#10). 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-27 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 (#6). 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-28 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 (#8). 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-27 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 (#7). Change subject: Index-only plan step 2: Added SplitOperator .. Index-only plan

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

2016-09-30 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 (#16). 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-09-30 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 (#2). Change subject: Full-text implementation step 1 .. Full-text implementation step

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

2016-09-30 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 (#11). 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-30 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Index-only plan step 2: Added SplitOperator .. Patch Set 11: (1 comment)

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

2016-09-30 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Index-only plan step 2: Added SplitOperator .. Patch Set 11: Addressed Jianfeng's comment: add a test case. -- To view, visit

Change in asterixdb[master]: .gitignore fix

2016-09-30 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: .gitignore fix .. .gitignore fix - Let .gitignore ignores the directories for the test results. Change-Id: If83bb49719bd41d5f71fefc785e4bcfae99d2d06

Change in asterixdb[master]: .gitignore fix

2016-09-30 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: .gitignore fix .. Patch Set 2: Oh ... -- To view, visit https://asterix-gerrit.ics.uci.edu/1235 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

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

2016-09-30 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 (#3). Change subject: Full-text implementation step 1 .. Full-text implementation step

Change in asterixdb[master]: .gitignore fix

2016-09-30 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1235 Change subject: .gitignore fix .. .gitignore fix - Let .gitignore ignores the directories for the test results. Change-Id:

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

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

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

2016-09-26 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 (#5). 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-29 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Index-only plan step 2: Added SplitOperator .. Patch Set 10: (6 comments) @Jianfeng: thanks for the comments. Regarding your first comments, I think we may need

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

2016-09-29 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Fixes for ASTERIXDB-1636 .. Patch Set 5: (1 comment)

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

2016-09-29 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Index-only plan step 2: Added SplitOperator .. Patch Set 10: PS: For your second comment, SplitOperatorDescriptor resides in Algebricks. So if we want to create

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

2016-09-29 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Index-only plan step 2: Added SplitOperator .. Patch Set 10: Got it. -- To view, visit https://asterix-gerrit.ics.uci.edu/1196 To unsubscribe, visit

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

2016-09-29 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Fixes for ASTERIXDB-1636 .. Patch Set 5: (1 comment)

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

2016-09-29 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Fixes for ASTERIXDB-1636 .. Patch Set 6: Code-Review+2 LGTM -- To view, visit https://asterix-gerrit.ics.uci.edu/1205 To unsubscribe, visit

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

2016-09-29 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Fixes for ASTERIXDB-1636 .. Patch Set 5: (2 comments)

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

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

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

2016-10-03 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 2: (5 comments)

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

2016-10-10 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 (#5). Change subject: Full-text implementation step 1 .. Full-text implementation step

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

2016-10-10 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 (#15). Change subject: Index-only plan step 2: Added SplitOperator ..

Change in asterixdb[master]: ASTERIXDB-1608, ASTERIXDB-1617 Match user query for nonpure ...

2016-10-10 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1608, ASTERIXDB-1617 Match user query for nonpure function calls .. Patch Set 11: One comment before going into the detail: I saw a lot of format

Change in asterixdb[master]: ASTERIXDB-1608, ASTERIXDB-1617 Match user query for nonpure ...

2016-10-10 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1608, ASTERIXDB-1617 Match user query for nonpure function calls .. Patch Set 12: We don't need to apply format changes to the unchanged parts. If you

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

2016-10-26 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 (#21). 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-25 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Full-text implementation step 1 .. Patch Set 15: (4 comments)

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

2016-10-25 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 (#16). Change subject: Full-text implementation step 1 .. Full-text implementation step

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

2016-10-27 Thread Taewoo Kim (Code Review)
Hello Ian Maxon, 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 (#18). Change subject: Full-text implementation step 1 .. Full-text

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

2016-10-27 Thread Taewoo Kim (Code Review)
Hello Ian Maxon, 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 (#19). Change subject: Full-text implementation step 1 .. Full-text

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

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

Change in asterixdb[master]: ASTERIXDB-1727: Fix the issue with multiple aggregates in on...

2016-11-09 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1343 to look at the new patch set (#3). Change subject: ASTERIXDB-1727: Fix the issue with multiple aggregates in one group-by

Change in asterixdb[master]: ASTERIXDB-1566, ASTERIXDB-1733: Hash Group By and Hash Join ...

2016-11-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 (#24). Change subject: ASTERIXDB-1566, ASTERIXDB-1733: Hash Group By and Hash Join conform to the memory budget

Change in asterixdb[master]: ASTERIXDB-1566, ASTERIXDB-1733: Hash Group By and Hash Join ...

2016-11-09 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1566, ASTERIXDB-1733: Hash Group By and Hash Join conform to the memory budget .. Patch Set 24: @Yingyi: please do not review this yet. I'm still

Change in asterixdb[master]: ASTERIXDB-1566, ASTERIXDB-1733: Hash Group By and Hash Join ...

2016-11-09 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1566, ASTERIXDB-1733: Hash Group By and Hash Join conform to the memory budget .. Patch Set 23: (9 comments)

Change in asterixdb[master]: ASTERIXDB-1727: Fix the issue with multiple aggregates in on...

2016-11-07 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1343 to look at the new patch set (#2). Change subject: ASTERIXDB-1727: Fix the issue with multiple aggregates in one group-by

Change in asterixdb[master]: ASTERIXDB-1566, ASTERIXDB-1733: Hash Group By and Hash Join ...

2016-11-10 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 (#25). Change subject: ASTERIXDB-1566, ASTERIXDB-1733: Hash Group By and Hash Join conform to the memory budget

Change in asterixdb[master]: ASTERIXDB-1727: Fix an issue with multiple aggregates in one...

2016-11-10 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1343 to look at the new patch set (#5). Change subject: ASTERIXDB-1727: Fix an issue with multiple aggregates in one group-by

Change in asterixdb[master]: ASTERIXDB-1727: Fix an issue with multiple aggregates in one...

2016-11-10 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1343 to look at the new patch set (#6). Change subject: ASTERIXDB-1727: Fix an issue with multiple aggregates in one group-by

Change in asterixdb[master]: ASTERIXDB-1727: Fix an issue with multiple aggregates in one...

2016-11-10 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1343 to look at the new patch set (#7). Change subject: ASTERIXDB-1727: Fix an issue with multiple aggregates in one group-by

Change in asterixdb[master]: ASTERIXDB-1727: Fix an issue with multiple aggregates in one...

2016-11-10 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1727: Fix an issue with multiple aggregates in one group-by .. Patch Set 4: (1 comment)

Change in asterixdb[master]: ASTERIXDB-1727: Fix an issue with multiple aggregates in one...

2016-11-10 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: ASTERIXDB-1727: Fix an issue with multiple aggregates in one group-by .. ASTERIXDB-1727: Fix an issue with multiple aggregates in one group-by -

Change in asterixdb[master]: ASTERIXDB-1727: Fix the issue with multiple aggregates in on...

2016-11-10 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1343 to look at the new patch set (#4). Change subject: ASTERIXDB-1727: Fix the issue with multiple aggregates in one group-by

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

2016-11-04 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 26: Wenhai, can you publish your comments? I can't

Change in asterixdb[master]: ASTERIXDB-1727: Fix the issue with multiple aggregates in on...

2016-11-04 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1343 Change subject: ASTERIXDB-1727: Fix the issue with multiple aggregates in one group-by .. ASTERIXDB-1727: Fix the issue with multiple

Change in asterixdb[master]: Ignore analyze-only plugin on Eclipse

2016-10-19 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1305 Change subject: Ignore analyze-only plugin on Eclipse .. Ignore analyze-only plugin on Eclipse - Let the Eclipse ignore analyze-only

Change in asterixdb[master]: Ignore analyze-only plugin on Eclipse

2016-10-19 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: Ignore analyze-only plugin on Eclipse .. Ignore analyze-only plugin on Eclipse - Let the Eclipse ignore analyze-only plugin since there is no m2e

Change in asterixdb[master]: ASTERIXDB-1608, ASTERIXDB-1617 Match user query for nonpure ...

2016-10-14 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1608, ASTERIXDB-1617 Match user query for nonpure function calls .. Patch Set 13: Can we add a few execution test cases? And it would be nice if you can

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

2016-10-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 21: (10 comments) More questions on the

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

2016-10-17 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 (#11). Change subject: Full-text implementation step 1 .. Full-text implementation step

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

2016-10-17 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 (#10). Change subject: Full-text implementation step 1 .. Full-text implementation step

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

2016-10-24 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Full-text implementation step 1 .. Patch Set 14: (4 comments)

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

2016-10-24 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 (#15). Change subject: Full-text implementation step 1 .. Full-text implementation step

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

2016-10-21 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 (#12). Change subject: Full-text implementation step 1 .. Full-text implementation step

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

2016-10-21 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Full-text implementation step 1 .. Patch Set 11: (14 comments) Thanks Ian!

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

2016-10-21 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 (#13). Change subject: Full-text implementation step 1 .. Full-text implementation step

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

2016-10-24 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 (#14). Change subject: Full-text implementation step 1 .. Full-text implementation step

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

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

Change in asterixdb[master]: ASTERIXDB-1700: fixed multiple same type of index applicatio...

2016-10-20 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new patch set (#2). Change subject: ASTERIXDB-1700: fixed multiple same type of index application error on the same field .. ASTERIXDB-1700: fixed multiple same type of index application error on the

Change in asterixdb[master]: ASTERIXDB-1608, ASTERIXDB-1617 Match user query for nonpure ...

2016-10-20 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1608, ASTERIXDB-1617 Match user query for nonpure function calls .. Patch Set 18: (29 comments) 1) Let's not change the license header since the next

Change in asterixdb[master]: ASTERIXDB-1700: fixed multiple same type of index applicatio...

2016-10-20 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1307 to look at the new patch set (#3). Change subject: ASTERIXDB-1700: fixed multiple same type of index application error on the same field

Change in asterixdb[master]: ASTERIXDB-1700: fixed multiple same type of index applicatio...

2016-10-20 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1307 Change subject: ASTERIXDB-1700: fixed multiple same type of index application error on the same field .. ASTERIXDB-1700: fixed multiple

Change in asterixdb[master]: ASTERIXDB-1700: fixed multiple same type of index applicatio...

2016-10-20 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: ASTERIXDB-1700: fixed multiple same type of index application error on the same field .. ASTERIXDB-1700: fixed multiple same type of index application

Change in asterixdb[master]: ASTERIXDB-1566, ASTERIXDB-1733: Hash Group By and Hash Join ...

2016-11-12 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 (#27). Change subject: ASTERIXDB-1566, ASTERIXDB-1733: Hash Group By and Hash Join conform to the memory budget

Change in asterixdb[master]: ASTERIXDB-1566, ASTERIXDB-1733: Hash Group By and Hash Join ...

2016-11-21 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1566, ASTERIXDB-1733: Hash Group By and Hash Join conform to the memory budget .. Patch Set 29: (3 comments) Thanks Till for your comments.

Change in asterixdb[master]: ASTERIXDB-1736: Remove Grace Hash Join and Hybrid Hash Join ...

2016-11-21 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1353 to look at the new patch set (#2). Change subject: ASTERIXDB-1736: Remove Grace Hash Join and Hybrid Hash Join (not being used)

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

2016-11-22 Thread Taewoo Kim (Code Review)
Hello Ian Maxon, 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 (#24). Change subject: Full-text implementation step 1 .. Full-text

Change in asterixdb[master]: ASTERIXDB-1566, ASTERIXDB-1733: Hash Group By and Hash Join ...

2016-11-22 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 (#31). Change subject: ASTERIXDB-1566, ASTERIXDB-1733: Hash Group By and Hash Join conform to the memory budget

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

2016-11-22 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 (#5). Change subject: Index-only plan step 3: Top-down Select and Join transformation rule

Change in asterixdb[master]: ASTERIXDB-1566, ASTERIXDB-1733: Hash Group By and Hash Join ...

2016-11-21 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 (#30). Change subject: ASTERIXDB-1566, ASTERIXDB-1733: Hash Group By and Hash Join conform to the memory budget

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

2016-11-24 Thread Taewoo Kim (Code Review)
Hello Ian Maxon, 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 (#25). Change subject: Full-text implementation step 1 .. Full-text

<    1   2   3   4   5   6   7   8   >