Change in asterixdb[master]: ASTERIXDB-1877: Tokenizer in FullText Search fix

2017-04-07 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1668 Change subject: ASTERIXDB-1877: Tokenizer in FullText Search fix .. ASTERIXDB-1877: Tokenizer in FullText Search fix - Sets the

Change in asterixdb[master]: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required

2017-08-02 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1880 to look at the new patch set (#8). Change subject: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required ..

Change in asterixdb[master]: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required

2017-08-02 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1880 to look at the new patch set (#7). Change subject: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required ..

Change in asterixdb[master]: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required

2017-08-02 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1880 to look at the new patch set (#6). Change subject: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required ..

Change in asterixdb[master]: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required

2017-08-03 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1880 to look at the new patch set (#10). Change subject: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required ..

Change in asterixdb[master]: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required

2017-08-03 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1880 to look at the new patch set (#9). Change subject: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required ..

Change in asterixdb[master]: [ASTERIXDB-1984][COMP] probe-subtree init not required

2017-08-12 Thread Taewoo Kim (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1880 to look at the new patch set (#15). Change subject: [ASTERIXDB-1984][COMP] probe-subtree init not required

Change in asterixdb[master]: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-07-07 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#11). Change subject: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-07-07 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#10). Change subject: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: [ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-07-09 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#13). Change subject: [ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-07-08 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#12). Change subject: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: [WIP][ASTERIXDB-1984][COMP] proper fieldtype propagation

2017-07-15 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1880 Change subject: [WIP][ASTERIXDB-1984][COMP] proper fieldtype propagation .. [WIP][ASTERIXDB-1984][COMP] proper fieldtype propagation -

Change in asterixdb[master]: [WIP][ASTERIXDB-1984][COMP] proper fieldtype propagation

2017-07-15 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new patch set (#2). Change subject: [WIP][ASTERIXDB-1984][COMP] proper fieldtype propagation .. [WIP][ASTERIXDB-1984][COMP] proper fieldtype propagation - user model changes: no - storage format

Change in asterixdb[master]: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required

2017-07-17 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1880 to look at the new patch set (#5). Change subject: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required ..

Change in asterixdb[master]: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required

2017-07-17 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1880 to look at the new patch set (#4). Change subject: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required ..

Change in asterixdb[master]: [WIP][ASTERIXDB-1984][COMP] proper fieldtype propagation

2017-07-15 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1880 to look at the new patch set (#3). Change subject: [WIP][ASTERIXDB-1984][COMP] proper fieldtype propagation ..

Change in asterixdb[master]: [WIP][ASTERIXDB-1984][COMP] proper fieldtype propagation

2017-07-15 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [WIP][ASTERIXDB-1984][COMP] proper fieldtype propagation .. Patch Set 2: (6 comments)

Change in asterixdb[master]: [ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-07-03 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-1972][COMP][RT][TX] index-only plan .. Patch Set 1: WIP... Please do not review this. -- To view, visit https://asterix-gerrit.ics.uci.edu/1866 To

Change in asterixdb[master]: [ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-07-03 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1866 Change subject: [ASTERIXDB-1972][COMP][RT][TX] index-only plan .. [ASTERIXDB-1972][COMP][RT][TX] index-only plan - user model changes:

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

2017-07-04 Thread Taewoo Kim (Code Review)
Taewoo Kim has abandoned this change. Change subject: ASTERIXDB-1791: Add a sanity check to hashjoin partition reading .. Abandoned -- To view, visit https://asterix-gerrit.ics.uci.edu/1505 To unsubscribe, visit

Change in asterixdb[master]: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-07-04 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#3). Change subject: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: [ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-07-04 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#2). Change subject: [ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-07-05 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#5). Change subject: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-07-06 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#9). Change subject: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-07-06 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#8). Change subject: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-07-06 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#7). Change subject: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-07-05 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#6). Change subject: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-07-04 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#4). Change subject: [WIP][ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: Hyracks Unit tests for LSM Filters

2017-04-26 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Hyracks Unit tests for LSM Filters .. Patch Set 3: (1 comment) General comment: if we keep the original function or class and if it calls the new function

Change in asterixdb[master]: ASTERIXDB-1892: Sets a proper hash table cardinality during ...

2017-04-25 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1702 Change subject: ASTERIXDB-1892: Sets a proper hash table cardinality during hash-group by .. ASTERIXDB-1892: Sets a proper hash table

Change in asterixdb[master]: Hyracks Unit tests for LSM Filters

2017-04-28 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Hyracks Unit tests for LSM Filters .. Patch Set 9: (7 comments) One Q: is there a test for "filtered = true" case? All I saw was "filtered = false".

Change in asterixdb[master]: ASTERIXDB-1892: Sets a proper hash table cardinality during ...

2017-04-27 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1892: Sets a proper hash table cardinality during hash-group by .. Patch Set 3: (1 comment)

Change in asterixdb[master]: ASTERIXDB-1892: Sets a proper hash table cardinality during ...

2017-04-27 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1892: Sets a proper hash table cardinality during hash-group by .. Patch Set 3: (2 comments) Thanks Yingyi. Here is my reply.

Change in asterixdb[master]: Index-Only Plan Patch Set 4: Index SearchCursor adjustment

2017-04-25 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: Index-Only Plan Patch Set 4: Index SearchCursor adjustment .. Index-Only Plan Patch Set 4: Index SearchCursor adjustment - Modify the search cursors to

Change in asterixdb[master]: ASTERIXDB-1892: Sets a proper hash table cardinality during ...

2017-04-27 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1702 to look at the new patch set (#4). Change subject: ASTERIXDB-1892: Sets a proper hash table cardinality during hash-group by

Change in asterixdb[master]: ASTERIXDB-1892: Sets a proper hash table cardinality during ...

2017-04-27 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1702 to look at the new patch set (#5). Change subject: ASTERIXDB-1892: Sets a proper hash table cardinality during hash-group by

Change in asterixdb[master]: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required

2017-08-04 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required .. Patch Set 10: (4 comments) For the test cases, I will also keep the original cases so that we will

Change in asterixdb[master]: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required

2017-08-04 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required .. Patch Set 10: (1 comment)

Change in asterixdb[master]: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required

2017-08-04 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1880 to look at the new patch set (#11). Change subject: [WIP][ASTERIXDB-1984][COMP] probe-subtree init not required ..

Change in asterixdb[master]: [ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-08-05 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#14). Change subject: [ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: [ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-08-05 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-1972][COMP][RT][TX] index-only plan .. Patch Set 13: (104 comments)

Change in asterixdb[master]: [ASTERIXDB-1984][COMP] probe-subtree init not required

2017-08-05 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1880 to look at the new patch set (#12). Change subject: [ASTERIXDB-1984][COMP] probe-subtree init not required ..

Change in asterixdb[master]: [ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-08-05 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#15). Change subject: [ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: [ASTERIXDB-1984][COMP] probe-subtree init not required

2017-08-07 Thread Taewoo Kim (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1880 to look at the new patch set (#14). Change subject: [ASTERIXDB-1984][COMP] probe-subtree init not required

Change in asterixdb[master]: ASTERIXDB-1892: Sets a proper hash table cardinality during ...

2017-05-02 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: ASTERIXDB-1892: Sets a proper hash table cardinality during hash-group by .. ASTERIXDB-1892: Sets a proper hash table cardinality during hash-group by -

Change in asterixdb[master]: Expected Hash Table footprint size calculation fix

2017-05-11 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1738 Change subject: Expected Hash Table footprint size calculation fix .. Expected Hash Table footprint size calculation fix - Fix the

Change in asterixdb[master]: Expected Hash Table footprint size calculation fix

2017-05-11 Thread Taewoo Kim (Code Review)
Hello Yingyi Bu, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1738 to look at the new patch set (#3). Change subject: Expected Hash Table footprint size calculation fix ..

Change in asterixdb[master]: ASTERIXDB-1789: fix an accessor that is used by Hash Table

2017-05-11 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1789: fix an accessor that is used by Hash Table .. Patch Set 1: As you can see, what this class is doing is fetching frameTupleAccessor for a tuple's

Change in asterixdb[master]: ASTERIXDB-1789: fix an accessor that is used by Hash Table

2017-05-11 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1734 to look at the new patch set (#2). Change subject: ASTERIXDB-1789: fix an accessor that is used by Hash Table ..

Change in asterixdb[master]: ASTERIXDB-1789: fix an accessor that is used by Hash Table

2017-05-10 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1734 Change subject: ASTERIXDB-1789: fix an accessor that is used by Hash Table .. ASTERIXDB-1789: fix an accessor that is used by Hash Table

Change in asterixdb[master]: ASTERIXDB-1789: fix an accessor that is used by Hash Table

2017-05-11 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: ASTERIXDB-1789: fix an accessor that is used by Hash Table .. ASTERIXDB-1789: fix an accessor that is used by Hash Table - Fix an accesor that is used by

Change in asterixdb[master]: Implemented LSM disk components alignment

2017-05-17 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Implemented LSM disk components alignment .. Patch Set 18: I think we need to consider a situation where a secondary index is built at a later time. For example,

Change in asterixdb[master]: Metadata Count Temp Fix

2017-05-17 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Metadata Count Temp Fix .. Patch Set 6: Code-Review+2 Thanks! Well done. -- To view, visit https://asterix-gerrit.ics.uci.edu/1729 To unsubscribe, visit

Change in asterixdb[master]: Change logical plan to apply filter from 2ndary index

2017-05-19 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Change logical plan to apply filter from 2ndary index .. Patch Set 7: (7 comments) General comments for each test: can you put a description on each test? What

Change in asterixdb[master]: Add query/aql endpoint.

2017-05-29 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Add query/aql endpoint. .. Patch Set 8: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1653 To unsubscribe, visit

Change in asterixdb[master]: Log filter values for redo during recovery

2017-06-08 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Log filter values for redo during recovery .. Patch Set 5: (7 comments) Thanks! A few thoughts

Change in asterixdb[master]: ASTERIXDB-1371 - Define new datatype 'geometry' user model c...

2017-06-15 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1371 - Define new datatype 'geometry' user model changes: Add new builtin type 'geometry' storage format changes: no interface changes: no .. Patch Set

Change in asterixdb[master]: ASTERIXDB-1371 - Define new datatype 'geometry' user model c...

2017-06-15 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1371 - Define new datatype 'geometry' user model changes: Add new builtin type 'geometry' storage format changes: no interface changes: no .. Patch Set

Change in asterixdb[master]: Hyracks Unit tests for LSM Filters

2017-04-30 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Hyracks Unit tests for LSM Filters .. Patch Set 10: (2 comments) Almost there. Just two comments. :-)

Change in asterixdb[master]: Hyracks Unit tests for LSM Filters

2017-05-01 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Hyracks Unit tests for LSM Filters .. Patch Set 10: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1704 To unsubscribe, visit

Change in asterixdb[master]: ASTERIXDB-1892: Sets a proper hash table cardinality during ...

2017-05-01 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1702 to look at the new patch set (#8). Change subject: ASTERIXDB-1892: Sets a proper hash table cardinality during hash-group by

Change in asterixdb[master]: ASTERIXDB-1892: Sets a proper hash table cardinality during ...

2017-05-01 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-1892: Sets a proper hash table cardinality during hash-group by .. Patch Set 7: (1 comment) Removed the header frame compaction.

Change in asterixdb[master]: Fix: Cursor can release entity lock twice in case of reconsile

2017-06-05 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Fix: Cursor can release entity lock twice in case of reconsile .. Patch Set 2: Abdullah, can you present a scenario? -- To view, visit

Change in asterixdb[master]: [ASTERIXDB-1984][COMP] probe-subtree init not required

2017-09-15 Thread Taewoo Kim (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1880 to look at the new patch set (#17). Change subject: [ASTERIXDB-1984][COMP] probe-subtree init not required

Change in asterixdb[master]: [ASTERIXDB-1984][COMP] probe-subtree init not required

2017-09-16 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-1984][COMP] probe-subtree init not required .. Patch Set 17: (6 comments) Thanks Dmitry for the initial comments.

Change in asterixdb[master]: [ASTERIXDB-1984][COMP] probe-subtree init not required

2017-09-16 Thread Taewoo Kim (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1880 to look at the new patch set (#18). Change subject: [ASTERIXDB-1984][COMP] probe-subtree init not required

Change in asterixdb[master]: [ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-10-16 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#16). Change subject: [ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: [ASTERIXDB-1984][COMP] probe-subtree init not required

2017-09-28 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-1984][COMP] probe-subtree init not required .. Patch Set 3: @Dmitry: OK. Let's do that. Thanks! -- To view, visit

Change in asterixdb[master]: [ASTERIXDB-1984][COMP] probe-subtree init not required

2017-09-28 Thread Taewoo Kim (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2030 to look at the new patch set (#4). Change subject: [ASTERIXDB-1984][COMP] probe-subtree init not required

Change in asterixdb[master]: [ASTERIXDB-1984][COMP] probe-subtree init not required

2017-09-28 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: [ASTERIXDB-1984][COMP] probe-subtree init not required .. [ASTERIXDB-1984][COMP] probe-subtree init not required - user model changes: no - storage format

Change in asterixdb[master]: [ASTERIXDB-2119][COMP] Fix variable ordering of projects

2017-10-02 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2119][COMP] Fix variable ordering of projects .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2043 To unsubscribe,

Change in asterixdb[master]: [ASTERIXDB-1984][COMP] probe-subtree init not required

2017-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/2030 to look at the new patch set (#2). Change subject: [ASTERIXDB-1984][COMP] probe-subtree init not required ..

Change in asterixdb[master]: [ASTERIXDB-1984][COMP] probe-subtree init not required

2017-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/2030 to look at the new patch set (#3). Change subject: [ASTERIXDB-1984][COMP] probe-subtree init not required ..

Change in asterixdb[master]: [ASTERIXDB-1984][COMP] probe-subtree init not required

2017-09-27 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2030 Change subject: [ASTERIXDB-1984][COMP] probe-subtree init not required .. [ASTERIXDB-1984][COMP] probe-subtree init not required - user

Change in asterixdb[master]: [ASTERIXDB-1984][COMP] probe-subtree init not required

2017-09-27 Thread Taewoo Kim (Code Review)
Taewoo Kim has abandoned this change. Change subject: [ASTERIXDB-1984][COMP] probe-subtree init not required .. Abandoned Submitted a new patch set. -- To view, visit https://asterix-gerrit.ics.uci.edu/1880 To unsubscribe,

Change in asterixdb[master]: [ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-11-24 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#17). Change subject: [ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: [ASTERIXDB-1984][COMP] probe-subtree init not required - fix #2

2017-11-22 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: [ASTERIXDB-1984][COMP] probe-subtree init not required - fix #2 .. [ASTERIXDB-1984][COMP] probe-subtree init not required - fix #2 - user model changes:

Change in asterixdb[master]: [ASTERIXDB-2176] Fix deletion on LSMRTree

2017-11-30 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2176] Fix deletion on LSMRTree .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2188 To unsubscribe, visit

Change in asterixdb[master]: [ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-11-30 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-1972][COMP][RT][TX] index-only plan .. Patch Set 17: (16 comments)

Change in asterixdb[master]: [ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-11-30 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#18). Change subject: [ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: [ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-11-30 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#19). Change subject: [ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: [ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-11-30 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#20). Change subject: [ASTERIXDB-1972][COMP][RT][TX] index-only plan ..

Change in asterixdb[master]: [ASTERIXDB-1984][COMP] probe-subtree init not required - fix #2

2017-11-22 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-1984][COMP] probe-subtree init not required - fix #2 .. Patch Set 2: Code-Review+2 Forwarding Dmitry's +2 from patch set 1 (just rebased) -- To view,

Change in asterixdb[master]: [ASTERIXDB-2176] Fix deletion on LSMRTree

2017-11-29 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2176] Fix deletion on LSMRTree .. Patch Set 1: Code-Review+2 (1 comment) LGTM. Only one minor comment

Change in asterixdb[master]: [ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-12-15 Thread Taewoo Kim (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#28). Change subject: [ASTERIXDB-1972][COMP][RT][TX] index-only plan

Change in asterixdb[master]: [ASTERIXDB-1972][COMP][RT][TX] index-only plan

2017-12-19 Thread Taewoo Kim (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1866 to look at the new patch set (#29). Change subject: [ASTERIXDB-1972][COMP][RT][TX] index-only plan

Change in asterixdb[master]: [ASTERIXDB-1984][COMP] probe-subtree init not required - fix #2

2017-11-21 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2175 Change subject: [ASTERIXDB-1984][COMP] probe-subtree init not required - fix #2 .. [ASTERIXDB-1984][COMP] probe-subtree init not

Change in asterixdb[master]: [ASTERIXDB-2153][COMP][RT] Ensure the fulltext search option...

2017-11-03 Thread Taewoo Kim (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2116 to look at the new patch set (#6). Change subject: [ASTERIXDB-2153][COMP][RT] Ensure the fulltext search option is properly handled

Change in asterixdb[master]: [NO ISSUE][RT][IDX] Simplify index.createAccessor()

2017-11-06 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE][RT][IDX] Simplify index.createAccessor() .. Patch Set 6: (2 comments) @Mike.B: thanks!

Change in asterixdb[master]: [NO ISSUE][RT][IDX] Simplify index.createAccessor()

2017-11-06 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE][RT][IDX] Simplify index.createAccessor() .. Patch Set 6: (1 comment) @Abdullah: thanks for the comment. @Till: how's your thought on his comment?

Change in asterixdb[master]: [NO ISSUE][RT][IDX] Simplify index.createAccessor()

2017-11-06 Thread Taewoo Kim (Code Review)
Hello Anon. E. Moose #1000171, abdullah alamoudi, Till Westmann, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2120 to look at the new patch set (#7). Change subject: [NO ISSUE][RT][IDX] Simplify index.createAccessor()

Change in asterixdb[master]: [NO ISSUE][RT][IDX] Simplify index.createAccessor()

2017-11-06 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE][RT][IDX] Simplify index.createAccessor() .. Patch Set 7: @Till and @Abdullah: thanks for the point. :-) @Abdullah: I have fixed the method name based

Change in asterixdb[master]: [NO ISSUE][RT][IDX] Simplify index.createAccessor()

2017-11-02 Thread Taewoo Kim (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2120 to look at the new patch set (#2). Change subject: [NO ISSUE][RT][IDX] Simplify index.createAccessor()

Change in asterixdb[master]: [NO ISSUE][RT][IDX] Simplify index.createAccessor()

2017-11-02 Thread Taewoo Kim (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2120 to look at the new patch set (#3). Change subject: [NO ISSUE][RT][IDX] Simplify index.createAccessor()

Change in asterixdb[master]: [NO ISSUE][RT][IDX] Simplify index.createAccessor()

2017-11-02 Thread Taewoo Kim (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2120 to look at the new patch set (#4). Change subject: [NO ISSUE][RT][IDX] Simplify index.createAccessor()

Change in asterixdb[master]: [NO ISSUE][RT][IDX] simplify createAccessor()

2017-11-02 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE][RT][IDX] simplify createAccessor() .. Patch Set 1: (1 comment) Thanks Till!

Change in asterixdb[master]: [ASTERIXDB-2153][COMP][RT] Ensure the fulltext search option...

2017-11-02 Thread Taewoo Kim (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2116 to look at the new patch set (#4). Change subject: [ASTERIXDB-2153][COMP][RT] Ensure the fulltext search option is properly handled

Change in asterixdb[master]: [ASTERIXDB-2153][COMP][RT] Ensure the fulltext search option...

2017-11-07 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2153][COMP][RT] Ensure the fulltext search option is properly handled .. Patch Set 9: Code-Review+2 Forwarding Dmitry's +2 from the previous patch set

Change in asterixdb[master]: [ASTERIXDB-2153][COMP][RT] Ensure the fulltext search option...

2017-11-07 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: [ASTERIXDB-2153][COMP][RT] Ensure the fulltext search option is properly handled .. [ASTERIXDB-2153][COMP][RT] Ensure the fulltext search option is

Change in asterixdb[master]: [NO ISSUE][COMP] Function references cleanup

2017-11-08 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE][COMP] Function references cleanup .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2130 To unsubscribe, visit

<    1   2   3   4   5   6   7   8   >