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-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 unsubscrib

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 https:/

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]: 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 - Second

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 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 https://asterix-gerrit.ics.uci.e

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, visit

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 compacted

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 https://ast

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

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

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 https:/

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 https://asterix-g

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-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-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-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) https://asterix-gerrit.ics.uci.edu/#/c/1144/3/hyracks-fullstack/hyracks/hyrack

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 (#4). 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) https://asterix-gerrit.ics.uci.edu/#/c/1119/8/asterixdb/asterix-algebra/src/m

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: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/1119/8//COMMIT_MSG Commit Message: Lin

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 https://asterix-gerrit

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 submitted this change and it was merged. Change subject: ASTERIXDB-1628: Fixed an issue in External Hash Group by .. ASTERIXDB-1628: Fixed an issue in External Hash Group by - The number of partitions in Externa

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) https://asterix-gerrit.ics.uci.edu/#/c/1119/10/asterixdb/asterix-algebra/sr

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-1081, ASTERIXDB-1086, ASTERIXDB-1246: proper multi...

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

Change in asterixdb[master]: Fulltext search initial implementation

2016-09-10 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Fulltext search initial implementation .. Patch Set 14: (84 comments) https://asterix-gerrit.ics.uci.edu/#/c/989/14/asterixdb/asterix-algebra/src/main/java/org/apa

Change in asterixdb[master]: Fulltext search initial implementation

2016-09-11 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 .. Fulltex

Change in asterixdb[master]: Full-text 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 (#16). Change subject: Full-text search initial implementation .. Full-t

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

2016-09-12 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 11: (3 comments) https://asterix-gerrit.ics.uci.edu/#/c/1119/11/asterixdb/asterix-algebra/s

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) https://asterix-gerrit.ics.uci.edu

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 htt

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-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]: 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]: 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 .. Fulltex

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! https://asterix-gerrit.ics.uci.edu/#/c/1131/1/asterixdb/asterix-doc/src/site/m

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]: 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 impleme

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 impleme

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) https://asterix-gerrit.ics.uci.edu/#/c/1175/3/asterixdb/asterix-om/src/main/java/org/apache/aste

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 impleme

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 impleme

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 (#6). Change subject: Full-text search implementation step 1 .. Full-text search impleme

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 ht

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 impleme

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) https://asterix-gerrit.ics.uci.edu/#/c

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]: 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 https://asterix-gerrit.ics.uci.edu/setti

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) https://asterix-gerrit.ics.uci.edu/#/c/

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, visit

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: 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 (#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]: 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) https://asterix-gerrit.ics.uci.edu/#/c/1056/11/hyracks-

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) https://asterix-gerrit.ics.uci.edu/#/c/1073/17/h

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]: 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-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-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 vi

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 -- To view, visit https://asterix-gerrit.ics.uci

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 plan

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

2016-09-21 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1196 Change subject: Index-only plan step 2: Added SplitOperator .. Index-only plan step 2: Added SplitOperator - Introduced SplitOperator t

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 ste

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

2016-09-21 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) https://asterix-gerrit.ics.uci.edu/#/c/1196/2/hyracks-fullstack/algebricks/algebricks-core

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

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) https://asterix-gerrit.ics.uci.edu/#/c/1198/1/asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/

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 pat

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 ste

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]: 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) https://asterix-gerrit.ics.uci.edu/#/c/1214/1/hyracks-fullstack/hy

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 ste

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) https://asterix-gerrit.ics.uci.edu/#/c/1196/5/hyracks-fullstack/algebricks/algebricks-core

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 ste

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 ste

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

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 ste

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

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/1056 to look at the new patch set (#15). Change subject: ASTERIXDB-1566: Fixed External Hash Group By to conform to the memory budget

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 I

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 st

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) https://asterix-gerrit.ics.uci.edu/#/c/1205/5/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/mai

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 t

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 on

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 https://aster

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) https://asterix-gerrit.ics.uci.edu/#/c/1205/5/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/s

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) https://asterix-gerrit.ics.uci.edu/#/c/1205/5/hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main

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 https://asterix-gerrit

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]: 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 st

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 https://asterix-gerrit.ics.uci.edu/1

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) https://asterix-gerrit.ics.uci.edu/#/c/1196/11/hyracks-fullstack/algebricks/algebricks-cor

  1   2   3   4   5   6   7   8   9   >