Change in asterixdb[master]: [ASTERIXDB-2468] Extend CountVarToCountOneRule to support no...

2018-11-20 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2468] Extend CountVarToCountOneRule to support no group-by .. Patch Set 9: @Dmitry: Chen checked that the CountVartoCountOneRule only removes the

Change in asterixdb[master]: [ASTERIXDB-2454] Remove non-unique AQL tests (pt.1)

2018-10-29 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2454] Remove non-unique AQL tests (pt.1) .. Patch Set 4: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/3007 To unsubscribe, visit

Change in asterixdb[master]: [ASTERIXDB-2466][FUN] Implement window functions

2018-10-28 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2466][FUN] Implement window functions .. Patch Set 2: (12 comments) Ali already gave good comments. I have some questions. (1) What does

Change in asterixdb[master]: [ASTERIXDB-2454] Remove non-unique AQL tests

2018-10-17 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2454] Remove non-unique AQL tests .. Patch Set 7: Code-Review+2 LGTM. Thanks! -- To view, visit https://asterix-gerrit.ics.uci.edu/2979 To

Change in asterixdb[master]: [ASTERIXDB-2454][TEST] Remove most of AQL runtime test cases

2018-10-17 Thread Taewoo Kim (Code Review)
Taewoo Kim has abandoned this change. Change subject: [ASTERIXDB-2454][TEST] Remove most of AQL runtime test cases .. Abandoned Ian Maxon will take care of this issue. -- To view, visit

Change in asterixdb[master]: [NO ISSUE][COMP][RT] Enable multiway similarity joins

2018-10-09 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: [NO ISSUE][COMP][RT] Enable multiway similarity joins .. [NO ISSUE][COMP][RT] Enable multiway similarity joins - Enable the FuzzyJoinRule that transforms

Change in asterixdb[master]: [NO ISSUE][COMP][RT] Enable multiway similarity joins

2018-10-09 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE][COMP][RT] Enable multiway similarity joins .. Patch Set 100: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1076 To unsubscribe,

Change in asterixdb[master]: [NO ISSUE][COMP][RT] Enable multiway similarity joins

2018-10-08 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/1076 to look at the new patch set (#100). Change subject: [NO ISSUE][COMP][RT] Enable multiway similarity joins

Change in asterixdb[master]: [NO ISSUE][COMP][RT] Enable multiway similarity joins

2018-10-08 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/1076 to look at the new patch set (#99). Change subject: [NO ISSUE][COMP][RT] Enable multiway similarity joins

Change in asterixdb[master]: [NO ISSUE][COMP][RT] Enable multiway similarity joins

2018-10-08 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/1076 to look at the new patch set (#98). Change subject: [NO ISSUE][COMP][RT] Enable multiway similarity joins

Change in asterixdb[master]: [NO ISSUE][COMP][RT] Enable multiway similarity joins

2018-10-08 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/1076 to look at the new patch set (#97). Change subject: [NO ISSUE][COMP][RT] Enable multiway similarity joins

Change in asterixdb[master]: [NO ISSUE][COMP][RT] Enable multiway similarity joins

2018-10-08 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE][COMP][RT] Enable multiway similarity joins .. Patch Set 93: (1 comment)

Change in asterixdb[master]: [NO ISSUE][COMP][RT] Enable multiway similarity joins

2018-10-08 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/1076 to look at the new patch set (#96). Change subject: [NO ISSUE][COMP][RT] Enable multiway similarity joins

Change in asterixdb[master]: [NO ISSUE][COMP][RT] Enable multiway similarity joins

2018-10-03 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE][COMP][RT] Enable multiway similarity joins .. Patch Set 93: (1 comment)

Change in asterixdb[master]: [NO ISSUE][COMP][RT] Enable multiway similarity joins

2018-10-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/1076 to look at the new patch set (#94). Change subject: [NO ISSUE][COMP][RT] Enable multiway similarity joins

Change in asterixdb[master]: [NO ISSUE][COMP][RT] Enable multiway similarity joins

2018-10-03 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE][COMP][RT] Enable multiway similarity joins .. Patch Set 93: (7 comments) https://asterix-gerrit.ics.uci.edu/#/c/1076/5//COMMIT_MSG Commit Message:

Change in asterixdb[master]: [NO ISSUE][COMP][RT] Enable multiway similarity joins

2018-10-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/1076 to look at the new patch set (#93). Change subject: [NO ISSUE][COMP][RT] Enable multiway similarity joins

Change in asterixdb[master]: [NO ISSUE] User-defined Function Documentation update

2018-09-28 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE] User-defined Function Documentation update .. Patch Set 16: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2804 To unsubscribe,

Change in asterixdb[master]: [NO ISSUE] User-defined Function Documentation update

2018-09-27 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE] User-defined Function Documentation update .. Patch Set 15: Code-Review+1 -- To view, visit https://asterix-gerrit.ics.uci.edu/2804 To unsubscribe,

Change in asterixdb[master]: [NO ISSUE] User-defined Function Documentation update

2018-09-27 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE] User-defined Function Documentation update .. Patch Set 15: (1 comment) Just one question.

Change in asterixdb[master]: [NO ISSUE] User-defined Function Documentation update

2018-09-25 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE] User-defined Function Documentation update .. Patch Set 11: Can you rebase the patch set first? :-) -- To view, visit

Change in asterixdb[master]: [ASTERIXDB-2455][DOC] Deprecate AQL documentations

2018-09-24 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: [ASTERIXDB-2455][DOC] Deprecate AQL documentations .. [ASTERIXDB-2455][DOC] Deprecate AQL documentations - user model changes: no - storage format

Change in asterixdb[master]: [ASTERIXDB-2455][DOC] Deprecate AQL documentations

2018-09-24 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2977 Change subject: [ASTERIXDB-2455][DOC] Deprecate AQL documentations .. [ASTERIXDB-2455][DOC] Deprecate AQL documentations - user model

Change in asterixdb[master]: [ASTERIXDB-2437][COMP] Ensure the index-only plan on a compo...

2018-09-24 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: [ASTERIXDB-2437][COMP] Ensure the index-only plan on a composite index .. [ASTERIXDB-2437][COMP] Ensure the index-only plan on a composite index - user

Change in asterixdb[master]: [NO ISSUE][COMP][RT] Enable multiway similarity joins

2018-09-24 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/1076 to look at the new patch set (#91). Change subject: [NO ISSUE][COMP][RT] Enable multiway similarity joins

Change in asterixdb[master]: [ASTERIXDB-2454][TEST] Remove most of AQL runtime test cases

2018-09-23 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new patch set (#3). Change subject: [ASTERIXDB-2454][TEST] Remove most of AQL runtime test cases .. [ASTERIXDB-2454][TEST] Remove most of AQL runtime test cases - user model changes: no - storage

Change in asterixdb[master]: [ASTERIXDB-2454][TEST] Remove most of AQL runtime test cases

2018-09-23 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new patch set (#2). Change subject: [ASTERIXDB-2454][TEST] Remove most of AQL runtime test cases .. [ASTERIXDB-2454][TEST] Remove most of AQL runtime test cases - user model changes: no - storage

Change in asterixdb[master]: [ASTERIXDB-2437][COMP] Ensure the index-only plan on a compo...

2018-09-22 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2437][COMP] Ensure the index-only plan on a composite index .. Patch Set 3: (2 comments) @Dmitry: thanks. I have addressed your comments.

Change in asterixdb[master]: [ASTERIXDB-2437][COMP] Ensure the index-only plan on a compo...

2018-09-22 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2854 to look at the new patch set (#5). Change subject: [ASTERIXDB-2437][COMP] Ensure the index-only plan on a composite index

Change in asterixdb[master]: [ASTERIXDB-2437][COMP] Ensure the index-only plan on a compo...

2018-09-22 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2854 to look at the new patch set (#4). Change subject: [ASTERIXDB-2437][COMP] Ensure the index-only plan on a composite index

Change in asterixdb[master]: [ASTERIXDB-2440][COMP] Compiler failure with IN expression

2018-08-13 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2440][COMP] Compiler failure with IN expression .. Patch Set 1: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2890 To unsubscribe,

Change in asterixdb[master]: [ASTERIXDB-2438][COMP] Allow assigns when pushing limit into...

2018-08-08 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2438][COMP] Allow assigns when pushing limit into data scan .. Patch Set 1: Code-Review+2 (1 comment) Can you remove a trailing white space? Just a

Change in asterixdb[master]: [ASTERIXDB-2437][COMP] Ensure the index-only plan on a compo...

2018-08-07 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new patch set (#3). Change subject: [ASTERIXDB-2437][COMP] Ensure the index-only plan on a composite index .. [ASTERIXDB-2437][COMP] Ensure the index-only plan on a composite index - user model

Change in asterixdb[master]: [ASTERIXDB-2437][COMP] Ensure the index-only plan on a compo...

2018-08-07 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new patch set (#2). Change subject: [ASTERIXDB-2437][COMP] Ensure the index-only plan on a composite index .. [ASTERIXDB-2437][COMP] Ensure the index-only plan on a composite index - user model

Change in asterixdb[master]: [ASTERIXDB-2437][COMP] Ensure the index-only plan on a compo...

2018-08-07 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2854 Change subject: [ASTERIXDB-2437][COMP] Ensure the index-only plan on a composite index .. [ASTERIXDB-2437][COMP] Ensure the index-only

Change in asterixdb[master]: [ASTERIXDB-2434][COMP] Ensure execution mode before EnforceS...

2018-08-07 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2434][COMP] Ensure execution mode before EnforceStructuralPropertiesRule .. Patch Set 3: Code-Review+2 -- To view, visit

Change in asterixdb[master]: [ASTERIXDB-2428][COMP] Incorrect result with limit if offset...

2018-07-27 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2428][COMP] Incorrect result with limit if offset is negative .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2821

Change in asterixdb[master]: [ASTERIXDB-2424][COMP] No limit pushdown if multiple limit c...

2018-07-26 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2424][COMP] No limit pushdown if multiple limit clauses .. Patch Set 1: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2815 To

Change in asterixdb[master]: [ASTERIXDB-2420][COMP] Internal error with limit clause

2018-07-24 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2420][COMP] Internal error with limit clause .. Patch Set 1: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2805 To unsubscribe,

Change in asterixdb[master]: [ASTERIXDB-2417][COMP] Fix issues with LIMIT clause

2018-07-20 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2417][COMP] Fix issues with LIMIT clause .. Patch Set 3: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2788 To unsubscribe, visit

Change in asterixdb[master]: ASTERIXDB-2418][COMP] Fix NPE in SqlppExpressionToPlanTransl...

2018-07-19 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: ASTERIXDB-2418][COMP] Fix NPE in SqlppExpressionToPlanTranslator .. Patch Set 1: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2789 To

Change in asterixdb[master]: [ASTERIXDB-2411][COMP] Error when aggregate inside between o...

2018-07-13 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2411][COMP] Error when aggregate inside between operator .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2759 To

Change in asterixdb[master]: [ASTERIXDB-2380][COMP] Incorrect comparison of exchange oper...

2018-07-12 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2380][COMP] Incorrect comparison of exchange operators .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2754 To

Change in asterixdb[master]: [ASTERIXDB-2409][RT] Make the full-text search return entire...

2018-07-10 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: [ASTERIXDB-2409][RT] Make the full-text search return entire results .. [ASTERIXDB-2409][RT] Make the full-text search return entire results - user model

Change in asterixdb[master]: [ASTERIXDB-2409][RT] Make the full-text search return entire...

2018-07-09 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/2742 to look at the new patch set (#2). Change subject: [ASTERIXDB-2409][RT] Make the full-text search return entire results

Change in asterixdb[master]: [ASTERIXDB-2409][RT] Make the full-text search return entire...

2018-07-09 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2742 Change subject: [ASTERIXDB-2409][RT] Make the full-text search return entire results .. [ASTERIXDB-2409][RT] Make the full-text search

Change in asterixdb[master]: Index-only plan

2018-06-20 Thread Taewoo Kim (Code Review)
Taewoo Kim has restored this change. Change subject: Index-only plan .. Restored -- To view, visit https://asterix-gerrit.ics.uci.edu/744 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType:

Change in asterixdb[master]: [NO ISSUE][RT] Eliminate HashMap lookup during comparision

2018-05-22 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE][RT] Eliminate HashMap lookup during comparision .. Patch Set 1: Code-Review+2 LGTM too. -- To view, visit https://asterix-gerrit.ics.uci.edu/2655 To

Change in asterixdb[master]: Relax JAVA_HOME check for sample cluster

2018-05-09 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: Relax JAVA_HOME check for sample cluster .. Patch Set 3: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2638 To unsubscribe, visit

Change in asterixdb[master]: [NO ISSUE][STO] Eliminate S Lock for Disk Components

2018-05-08 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE][STO] Eliminate S Lock for Disk Components .. Patch Set 1: (1 comment) Thanks for taking care of this.

Change in asterixdb[master]: Index-only plan

2018-05-08 Thread Taewoo Kim (Code Review)
Taewoo Kim has abandoned this change. Change subject: Index-only plan .. Abandoned -- To view, visit https://asterix-gerrit.ics.uci.edu/744 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Change in asterixdb[master]: [ASTERIXDB-2344] LIMIT pushdown for primary index

2018-05-01 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2344] LIMIT pushdown for primary index .. Patch Set 17: Code-Review+1 LGTM. I only have a few minor comments. #1. Do we want to keep the optimizer

Change in asterixdb[master]: [ASTERIXDB-2374][RT] Index-only plan on B+Tree disk components

2018-04-27 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: [ASTERIXDB-2374][RT] Index-only plan on B+Tree disk components .. [ASTERIXDB-2374][RT] Index-only plan on B+Tree disk components - user model changes: no

Change in asterixdb[master]: [ASTERIXDB-2374][RT] Index-only plan on B+Tree disk components

2018-04-27 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/2623 to look at the new patch set (#2). Change subject: [ASTERIXDB-2374][RT] Index-only plan on B+Tree disk components

Change in asterixdb[master]: [ASTERIXDB-2374][RT] Index-only plan on B+Tree disk components

2018-04-27 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2623 Change subject: [ASTERIXDB-2374][RT] Index-only plan on B+Tree disk components .. [ASTERIXDB-2374][RT] Index-only plan on B+Tree disk

Change in asterixdb[master]: [ASTERIXDB-2356][SQL] Syntax error when parsing CASE expression

2018-04-19 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2356][SQL] Syntax error when parsing CASE expression .. Patch Set 2: Code-Review+2 (1 comment) LGTM - only one minor comment

Change in asterixdb[master]: [ASTERIXDB-2349][SITE] Revise fulltext and similarity docume...

2018-04-11 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: [ASTERIXDB-2349][SITE] Revise fulltext and similarity documentation .. [ASTERIXDB-2349][SITE] Revise fulltext and similarity documentation - user model

Change in asterixdb[master]: [ASTERIXDB-2349][SITE] Revise fulltext and similarity docume...

2018-04-04 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new patch set (#2). Change subject: [ASTERIXDB-2349][SITE] Revise fulltext and similarity documentation .. [ASTERIXDB-2349][SITE] Revise fulltext and similarity documentation - user model changes: no

Change in asterixdb[master]: [ASTERIXDB-2349][SITE] Revise fulltext and similarity docume...

2018-04-04 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2567 Change subject: [ASTERIXDB-2349][SITE] Revise fulltext and similarity documentation .. [ASTERIXDB-2349][SITE] Revise fulltext and

Change in asterixdb[master]: [ASTERIXDB-2344] LIMIT pushdown for primary index

2018-04-03 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2344] LIMIT pushdown for primary index .. Patch Set 5: (13 comments) Overall, it looks good.

Change in asterixdb[master]: [ASTERIXDB-2339] Add a new inverted index merge cursor

2018-03-24 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2339] Add a new inverted index merge cursor .. Patch Set 5: (3 comments) Generally, how opType is used in the bulk loader?

Change in asterixdb[master]: [ASTERIXDB-2338][RT] Allow concurrent accesses to an inverte...

2018-03-23 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: [ASTERIXDB-2338][RT] Allow concurrent accesses to an inverted list .. [ASTERIXDB-2338][RT] Allow concurrent accesses to an inverted list - user model

Change in asterixdb[master]: [ASTERIXDB-2338][RT] Allow concurrent accesses to an inverte...

2018-03-22 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2338][RT] Allow concurrent accesses to an inverted list .. Patch Set 1: (1 comment) Thanks!!!

Change in asterixdb[master]: [ASTERIXDB-2338][RT] Allow concurrent accesses to an inverte...

2018-03-22 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2517 to look at the new patch set (#2). Change subject: [ASTERIXDB-2338][RT] Allow concurrent accesses to an inverted list

Change in asterixdb[master]: [ASTERIXDB-2338][RT] Allow concurrent accesses to an inverte...

2018-03-22 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2517 Change subject: [ASTERIXDB-2338][RT] Allow concurrent accesses to an inverted list .. [ASTERIXDB-2338][RT] Allow concurrent accesses to

Change in asterixdb[master]: [NO ISSUE][COMP][RT] Rename index-only test case directory

2018-03-10 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2472 to look at the new patch set (#2). Change subject: [NO ISSUE][COMP][RT] Rename index-only test case directory .. [NO

Change in asterixdb[master]: [NO ISSUE][RT] no latching on an inverted list during merge

2018-03-01 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: [NO ISSUE][RT] no latching on an inverted list during merge .. [NO ISSUE][RT] no latching on an inverted list during merge - user-model changes: no -

Change in asterixdb[master]: [ASTERIXDB-2307][COMP] Incorrect result with quantified expr...

2018-03-01 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2307][COMP] Incorrect result with quantified expression .. Patch Set 2: Code-Review+2 LGTM! Only one minor comment: it would be wonderful if you could

Change in asterixdb[master]: [NO ISSUE][RT] no latching on an inverted list during merge

2018-03-01 Thread Taewoo Kim (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2444 to look at the new patch set (#2). Change subject: [NO ISSUE][RT] no latching on an inverted list during merge ..

Change in asterixdb[master]: [NO ISSUE][RT] no latching on an inverted list during merge

2018-03-01 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2444 Change subject: [NO ISSUE][RT] no latching on an inverted list during merge .. [NO ISSUE][RT] no latching on an inverted list during

Change in asterixdb[master]: [ASTERIXDB-2302][COMP] Incorrect result with non-enforced index

2018-02-26 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2302][COMP] Incorrect result with non-enforced index .. Patch Set 1: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2425 To

Change in asterixdb[master]: [ASTERIXDB-2296][COMP] proper handling of an optional subfie...

2018-02-22 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: [ASTERIXDB-2296][COMP] proper handling of an optional subfield type .. [ASTERIXDB-2296][COMP] proper handling of an optional subfield type - user-model

Change in asterixdb[master]: [NO ISSUE] Fix template, again

2018-02-22 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE] Fix template, again .. Patch Set 1: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2420 To unsubscribe, visit

Change in asterixdb[master]: [ASTERIXDB-2296][COMP] proper handling of an optional subfie...

2018-02-22 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/2419 to look at the new patch set (#3). Change subject: [ASTERIXDB-2296][COMP] proper handling of an optional subfield type

Change in asterixdb[master]: [ASTERIXDB-2296][COMP] proper handling of an optional subfie...

2018-02-22 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2296][COMP] proper handling of an optional subfield type .. Patch Set 1: (3 comments) Thanks!

Change in asterixdb[master]: [ASTERIXDB-2296][COMP] proper handling of an optional subfie...

2018-02-22 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2419 Change subject: [ASTERIXDB-2296][COMP] proper handling of an optional subfield type .. [ASTERIXDB-2296][COMP] proper handling of an

Change in asterixdb[master]: [NO ISSUE][RT] Ensure an inverted list cursor.close()

2018-02-21 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: [NO ISSUE][RT] Ensure an inverted list cursor.close() .. [NO ISSUE][RT] Ensure an inverted list cursor.close() - user model changes: no - storage format

Change in asterixdb[master]: [NO ISSUE][RT] Ensure an inverted list cursor.close()

2018-02-21 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE][RT] Ensure an inverted list cursor.close() .. Patch Set 4: (1 comment)

Change in asterixdb[master]: [NO ISSUE][RT] Ensure an inverted list cursor.close()

2018-02-21 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/2410 to look at the new patch set (#3). Change subject: [NO ISSUE][RT] Ensure an inverted list cursor.close()

Change in asterixdb[master]: [NO ISSUE][RT] Ensure an inverted list cursor.close()

2018-02-21 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/2410 to look at the new patch set (#4). Change subject: [NO ISSUE][RT] Ensure an inverted list cursor.close()

Change in asterixdb[master]: [NO ISSUE][RT] Ensure a cursor.close()

2018-02-20 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/2410 to look at the new patch set (#2). Change subject: [NO ISSUE][RT] Ensure a cursor.close() ..

Change in asterixdb[master]: [NO ISSUE][RT] Ensure a cursor.close()

2018-02-20 Thread Taewoo Kim (Code Review)
Taewoo Kim has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2410 Change subject: [NO ISSUE][RT] Ensure a cursor.close() .. [NO ISSUE][RT] Ensure a cursor.close() - user model changes: no - storage

Change in asterixdb[master]: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inv...

2018-02-19 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inverted index search .. [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inverted index

Change in asterixdb[master]: [NO ISSUE][RT] follow IFrameWriter protocol in SplitOperator...

2018-02-18 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE][RT] follow IFrameWriter protocol in SplitOperatorDescriptor .. Patch Set 4: Code-Review+2 LGTM. -- To view, visit

Change in asterixdb[master]: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inv...

2018-02-18 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/2251 to look at the new patch set (#37). Change subject: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inverted index search

Change in asterixdb[master]: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inv...

2018-02-16 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inverted index search .. Patch Set 35: Done. Thanks! -- To view, visit

Change in asterixdb[master]: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inv...

2018-02-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/2251 to look at the new patch set (#36). Change subject: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inverted index search

Change in asterixdb[master]: [NO ISSUE] Correct import of InvokeUtil in BufferCache

2018-02-16 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [NO ISSUE] Correct import of InvokeUtil in BufferCache .. Patch Set 1: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2396 To unsubscribe,

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

2018-02-16 Thread Taewoo Kim (Code Review)
Taewoo Kim has submitted this change and it was merged. Change subject: [ASTERIXDB-1972][COMP][RT][TX] index-only plan .. [ASTERIXDB-1972][COMP][RT][TX] index-only plan - user model changes: no - storage format changes: no -

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

2018-02-16 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 66: Code-Review+2 Forwarding Dmitry's +2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1866 To

Change in asterixdb-bad[master]: [ASTERIXDB-1972] Coordinated Change

2018-02-16 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-1972] Coordinated Change .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2376 To unsubscribe, visit

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

2018-02-15 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 65: (2 comments) Done!

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

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

Change in asterixdb[master]: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inv...

2018-02-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/2251 to look at the new patch set (#35). Change subject: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inverted index search

Change in asterixdb[master]: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inv...

2018-02-14 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/2251 to look at the new patch set (#34). Change subject: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inverted index search

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

2018-02-14 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 (#64). Change subject: [ASTERIXDB-1972][COMP][RT][TX] index-only plan

Change in asterixdb[master]: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inv...

2018-02-14 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/2251 to look at the new patch set (#33). Change subject: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inverted index search

Change in asterixdb[master]: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inv...

2018-02-14 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/2251 to look at the new patch set (#32). Change subject: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inverted index search

Change in asterixdb[master]: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inv...

2018-02-14 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/2251 to look at the new patch set (#31). Change subject: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inverted index search

Change in asterixdb[master]: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inv...

2018-02-14 Thread Taewoo Kim (Code Review)
Taewoo Kim has posted comments on this change. Change subject: [ASTERIXDB-2083][COMP][RT][IDX][SITE] Budget-Constrained Inverted index search .. Patch Set 30: (10 comments) Thanks Abdullah.

  1   2   3   4   5   6   7   8   >