Change in asterixdb[master]: [ASTERIXDB-2249][API] Add Result Max Reads to API

2018-01-16 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2249][API] Add Result Max Reads to API .. Patch Set 1: (2 comments) https://asterix-gerrit.ics.uci.edu/#/c/2292/1/asterixdb/asterix-app/src/main/java

Change in asterixdb[master]: [ASTERIXDB-2231][STO] Seperate primary op tracker for each p...

2018-01-07 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2231][STO] Seperate primary op tracker for each partition .. Patch Set 11: (2 comments) just a nit .. https://asterix-gerrit.ics.uci.edu/#/c/2263/1

Change in asterixdb[master]: [NO ISSUE][TX] Remove Unused Fields in LogRecord

2018-01-07 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][TX] Remove Unused Fields in LogRecord .. Patch Set 1: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/2269/1//COMMIT_MSG Commit Message: PS1, Lin

Change in asterixdb[master]: [ASTERIXDB-2228][CLUS] IFaultToleranceStrategy -> INcLifecyc...

2018-01-07 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2228][CLUS] IFaultToleranceStrategy -> INcLifecycleCoordinator .. Patch Set 1: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/22

Change in asterixdb[master]: [ASTERIXDB-2195][REPL] Replace Static Replication

2018-01-04 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2195][REPL] Replace Static Replication .. Patch Set 6: Code-Review+2 Feel free to forward the +2 after addressing comments or to file issues. -- To

Change in asterixdb[master]: [ASTERIXDB-2195][REPL] Replace Static Replication

2018-01-04 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2195][REPL] Replace Static Replication .. Patch Set 6: (3 comments) https://asterix-gerrit.ics.uci.edu/#/c/2252/6/asterixdb/asterix-app/src/main/java

Change in asterixdb[master]: [NO ISSUE] SQL++ doc updates

2018-01-02 Thread Till Westmann (Code Review)
Till Westmann has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2257 Change subject: [NO ISSUE] SQL++ doc updates .. [NO ISSUE] SQL++ doc updates Change-Id: I1aeeff79b94136f9791f0e2554f1161b8cbb5abb ---

Change in asterixdb[master]: [ASTERIXDB-2204][STO] Fix the IIndexCursor interface

2018-01-02 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2204][STO] Fix the IIndexCursor interface .. Patch Set 11: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2238 To unsubscribe, vi

Change in asterixdb[master]: [ASTERIXDB-2180][FUN] Provent dropping of entities used by f...

2018-01-02 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2180][FUN] Provent dropping of entities used by functions .. Patch Set 3: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/2253/3//COMMIT_MSG Comm

Change in asterixdb[master]: [ASTERIXDB-2205][STO] Maintain includeMutableComponent corre...

2017-12-30 Thread Till Westmann (Code Review)
Till Westmann has submitted this change and it was merged. Change subject: [ASTERIXDB-2205][STO] Maintain includeMutableComponent correctly .. [ASTERIXDB-2205][STO] Maintain includeMutableComponent correctly - user model change

Change in asterixdb[master]: [ASTERIXDB-1952][TX][IDX] Filter logs pt.2

2017-12-28 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-1952][TX][IDX] Filter logs pt.2 .. Patch Set 23: (6 comments) https://asterix-gerrit.ics.uci.edu/#/c/1857/23//COMMIT_MSG Commit Message: PS23, Line

Change in asterixdb[master]: [ASTERIXDB-2205][STO] Maintain includeMutableComponent corre...

2017-12-24 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2205][STO] Maintain includeMutableComponent correctly .. Patch Set 6: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2243 To unsu

Change in asterixdb[master]: [ASTERIXDB-2205][STO] Maintain includeMutableComponent corre...

2017-12-21 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2205][STO] Maintain includeMutableComponent correctly .. Patch Set 4: (2 comments) https://asterix-gerrit.ics.uci.edu/#/c/2243/4/hyracks-fullstack/hy

Change in asterixdb[master]: [NO ISSUE][OTR] Dynamically Determine Default Config File

2017-12-19 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][OTR] Dynamically Determine Default Config File .. Patch Set 3: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2239 To unsubscribe

Change in asterixdb[master]: [NO ISSUE][TEST] Allow Diagnostic files to be run for failin...

2017-12-19 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][TEST] Allow Diagnostic files to be run for failing tests .. Patch Set 17: Could we add a test that tests and demonstrates the feature? (Especially if

Change in asterixdb[master]: [ASTERIXDB-2193][SQLPP] Fix dataverse resolution in functions

2017-12-19 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2193][SQLPP] Fix dataverse resolution in functions .. Patch Set 2: Code-Review+2 (1 comment) Looks good - just one comment that could be addressed be

Change in asterixdb[master]: [NO ISSUE][RT] Remove Invalid Clear Queued Jobs API

2017-12-19 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][RT] Remove Invalid Clear Queued Jobs API .. Patch Set 1: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2237 To unsubscribe, visi

Change in asterixdb[master]: [ASTERIXDB-1962][COMP][RT] Remove FieldAccessNested

2017-12-15 Thread Till Westmann (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1858 to look at the new patch set (#3). Change subject: [ASTERIXDB-1962][COMP][RT] Remove FieldAccessNested .. [ASTERIXDB-

Change in asterixdb[master]: [ASTERIXDB-2124] Throw NPE early

2017-12-14 Thread Till Westmann (Code Review)
Till Westmann has submitted this change and it was merged. Change subject: [ASTERIXDB-2124] Throw NPE early .. [ASTERIXDB-2124] Throw NPE early Change-Id: I84a458d2293450bce973788ac84b6fbc4c993a4a Reviewed-on: https://asterix-g

Change in asterixdb[master]: [ASTERIXDB-2198][REPL] Introduce Dynamic Replica Placement

2017-12-13 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2198][REPL] Introduce Dynamic Replica Placement .. Patch Set 6: Code-Review+2 Looks good to me, but I don't understand the contrib failure ... -- To

Change in asterixdb[master]: [ASTERIXDB-2195][REPL] Introduce Dynamic Replica Placement

2017-12-13 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2195][REPL] Introduce Dynamic Replica Placement .. Patch Set 4: (3 comments) https://asterix-gerrit.ics.uci.edu/#/c/2215/4/asterixdb/asterix-app/src/

Change in asterixdb[master]: [ASTERIXDB-2197][FAIL] Abort Job on Failures in NotifyTaskCo...

2017-12-13 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2197][FAIL] Abort Job on Failures in NotifyTaskCompleteWork .. Patch Set 1: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2218

Change in asterixdb[master]: [ASTERIXDB-2198][CLUS] Introduce NodeJobTracker

2017-12-13 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2198][CLUS] Introduce NodeJobTracker .. Patch Set 1: Code-Review+2 (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/2220/1//COMMIT_MSG Commit Messa

Change in asterixdb[master]: [ASTERIXDB-2170][SQL] Fix resolution order of implicit field...

2017-12-12 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2170][SQL] Fix resolution order of implicit field access .. Patch Set 7: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2207 To

Change in asterixdb[master]: [ASTERIXDB-2194][COMP] Introduce datasource functions

2017-12-11 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2194][COMP] Introduce datasource functions .. Patch Set 6: (10 comments) https://asterix-gerrit.ics.uci.edu/#/c/2216/6/asterixdb/asterix-algebra/src/

Change in asterixdb[master]: [ASTERIXDB-2170][SQL] Fix resolution order of implicit field...

2017-12-11 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2170][SQL] Fix resolution order of implicit field access .. Patch Set 6: (11 comments) https://asterix-gerrit.ics.uci.edu/#/c/2207/6//COMMIT_MSG Com

Change in asterixdb[master]: [NO ISSUE][API] Add Result Location API

2017-12-10 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][API] Add Result Location API .. Patch Set 13: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/1742/13//COMMIT_MSG Commit Message: PS13, Line 7: N

Change in asterixdb[master]: [NO ISSUE][REPL] Introduce Dynamic Replica Placement

2017-12-10 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][REPL] Introduce Dynamic Replica Placement .. Patch Set 3: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/2215/3//COMMIT_MSG Commit Message: PS3,

Change in asterixdb[master]: [NO ISSUE][COMP] Introduce datasource functions

2017-12-10 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][COMP] Introduce datasource functions .. Patch Set 5: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/2216/5//COMMIT_MSG Commit Message: PS5, Line

Change in asterixdb[master]: [ASTERIXDB-2190][API] fix timeout behavior

2017-12-09 Thread Till Westmann (Code Review)
Till Westmann has submitted this change and it was merged. Change subject: [ASTERIXDB-2190][API] fix timeout behavior .. [ASTERIXDB-2190][API] fix timeout behavior - user model changes: no - storage format changes: no - interfa

Change in asterixdb[master]: [ASTERIXDB-2190][API] fix timeout behavior

2017-12-08 Thread Till Westmann (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, Murtadha Hubail, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2199 to look at the new patch set (#7). Change subject: [ASTERIXDB-2190][API] fix timeout behavior ..

Change in asterixdb[master]: [NO ISSUE] Remove deprecated uses of ObjectUtils

2017-12-08 Thread Till Westmann (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2205 to look at the new patch set (#5). Change subject: [NO ISSUE] Remove deprecated uses of ObjectUtils .

Change in asterixdb[master]: [NO ISSUE] Remove deprecated uses of ObjectUtils

2017-12-08 Thread Till Westmann (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2205 to look at the new patch set (#4). Change subject: [NO ISSUE] Remove deprecated uses of ObjectUtils .

Change in asterixdb[master]: [NO ISSUE] Fix duplicate -Xmx args being passed

2017-12-08 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE] Fix duplicate -Xmx args being passed .. Patch Set 5: And as a result of fixing this tests fail because there's not enough memory ... -- To view, vis

Change in asterixdb[master]: [NO ISSUE] Remove deprecated uses of ObjectUtils

2017-12-08 Thread Till Westmann (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2205 to look at the new patch set (#3). Change subject: [NO ISSUE] Remove deprecated uses of ObjectUtils .. [NO ISSUE] Rem

Change in asterixdb[master]: [NO ISSUE] Remove deprecated uses of ObjectUtils

2017-12-07 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE] Remove deprecated uses of ObjectUtils .. Patch Set 2: (6 comments) https://asterix-gerrit.ics.uci.edu/#/c/2205/1/asterixdb/asterix-lang-common/src/ma

Change in asterixdb[master]: [NO ISSUE] Remove deprecated uses of ObjectUtils

2017-12-07 Thread Till Westmann (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2205 to look at the new patch set (#2). Change subject: [NO ISSUE] Remove deprecated uses of ObjectUtils .. [NO ISSUE] Rem

Change in asterixdb[master]: [ASTERIXDB-2190][API] fix timeout behavior

2017-12-07 Thread Till Westmann (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2199 to look at the new patch set (#6). Change subject: [ASTERIXDB-2190][API] fix timeout behavior ...

Change in asterixdb[master]: [ASTERIXDB-2190][API] fix timeout behavior

2017-12-07 Thread Till Westmann (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2199 to look at the new patch set (#5). Change subject: [ASTERIXDB-2190][API] fix timeout behavior ...

Change in asterixdb[master]: [NO ISSUE] Fix duplicate -Xmx args being passed

2017-12-07 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE] Fix duplicate -Xmx args being passed .. Patch Set 5: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2206 To unsubscribe, visit ht

Change in asterixdb[master]: [NO ISSUE] Fix duplicate -Xmx args being passed

2017-12-07 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE] Fix duplicate -Xmx args being passed .. Patch Set 4: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2206 To unsubscribe, visit ht

Change in asterixdb[master]: [NO ISSUE] Remove deprecated uses of ObjectUtils

2017-12-07 Thread Till Westmann (Code Review)
Till Westmann has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2205 Change subject: [NO ISSUE] Remove deprecated uses of ObjectUtils .. [NO ISSUE] Remove deprecated uses of ObjectUtils - user model cha

Change in asterixdb[master]: [NO ISSUE][STO] Introduce Index Checkpoints

2017-12-07 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][STO] Introduce Index Checkpoints .. Patch Set 14: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2200 To unsubscribe, visit https

Change in asterixdb[master]: [NO ISSUE][STO] Introduce Index Checkpoints

2017-12-06 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][STO] Introduce Index Checkpoints .. Patch Set 11: (5 comments) https://asterix-gerrit.ics.uci.edu/#/c/2200/11//COMMIT_MSG Commit Message: PS11, Line

Change in asterixdb[master]: [NO ISSUE][STO] Introduce Index Checkpoints

2017-12-05 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][STO] Introduce Index Checkpoints .. Patch Set 7: (4 comments) https://asterix-gerrit.ics.uci.edu/#/c/2200/7/asterixdb/asterix-common/src/main/java/or

Change in asterixdb[master]: [NO ISSUE][CLUS] Allow Extension of cc application context

2017-12-05 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][CLUS] Allow Extension of cc application context .. Patch Set 7: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2192 To unsubscrib

Change in asterixdb[master]: [NO ISSUE][TEST] Add NC Storage API Test

2017-12-03 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][TEST] Add NC Storage API Test .. Patch Set 4: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/2195/4/asterixdb/asterix-app/src/main/java/org/apach

Change in asterixdb[master]: [WIP] fix timeout behavior

2017-12-03 Thread Till Westmann (Code Review)
Till Westmann has uploaded a new patch set (#2). Change subject: [WIP] fix timeout behavior .. [WIP] fix timeout behavior Change-Id: Ide0515dc8ef9f8c295e1dc2ffde297100634060a --- M asterixdb/asterix-app/src/main/java/org/apache

Change in asterixdb[master]: [WIP] fix timeout behavior

2017-12-03 Thread Till Westmann (Code Review)
Till Westmann has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2199 Change subject: [WIP] fix timeout behavior .. [WIP] fix timeout behavior Change-Id: Ide0515dc8ef9f8c295e1dc2ffde297100634060a --- M

Change in asterixdb[master]: [NO ISSUE] Remove duplicate jackson-databind

2017-12-03 Thread Till Westmann (Code Review)
Till Westmann has submitted this change and it was merged. Change subject: [NO ISSUE] Remove duplicate jackson-databind .. [NO ISSUE] Remove duplicate jackson-databind - user model changes: no - storage format changes: no - int

Change in asterixdb[master]: [NO ISSUE][CLUS] Allow Extension of cc application context

2017-12-01 Thread Till Westmann (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2192 to look at the new patch set (#4). Change subject: [NO ISSUE][CLUS] Allow Extension of cc application context ...

Change in asterixdb[master]: [NO ISSUE] Remove duplicate jackson-databind

2017-12-01 Thread Till Westmann (Code Review)
Till Westmann has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2198 Change subject: [NO ISSUE] Remove duplicate jackson-databind .. [NO ISSUE] Remove duplicate jackson-databind - user model changes: no

Change in asterixdb[master]: [NO ISSUE][API] Add NC Storage API

2017-12-01 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Storage API .. Patch Set 5: Code-Review+2 (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/2190/5/asterixdb/asterix-common/src/main/jav

Change in asterixdb[master]: [ASTERIXDB-2182][SQL] Change merge policy syntax

2017-12-01 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2182][SQL] Change merge policy syntax .. Patch Set 6: Code-Review+2 Forwarding Abdullah's +2 after rebase. -- To view, visit https://asterix-gerrit.

Change in asterixdb[master]: [NO ISSUE][API] Add NC Storage API

2017-11-30 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][API] Add NC Storage API .. Patch Set 4: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/2190/4/asterixdb/asterix-common/src/main/java/org/apache/a

Change in asterixdb[master]: [ASTERIXDB-2182][SQL] Change merge policy syntax

2017-11-30 Thread Till Westmann (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2183 to look at the new patch set (#5). Change subject: [ASTERIXDB-2182][SQL] Change merge policy syntax .. [ASTERIXDB-218

Change in asterixdb[master]: [ASTERIXDB-2182][SQL] Change merge policy syntax

2017-11-30 Thread Till Westmann (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2183 to look at the new patch set (#4). Change subject: [ASTERIXDB-2182][SQL] Change merge policy syntax .. [ASTERIXDB-218

Change in asterixdb[master]: [ASTERIXDB-2148][FUN] Add init parameter for external UDF

2017-11-22 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2148][FUN] Add init parameter for external UDF .. Patch Set 5: (4 comments) Hmm, why do the parameters need to transported to so many places. Couldn'

Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent

2017-11-20 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit ht

Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent

2017-11-20 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: (3 comments) https://asterix-gerrit.ics.uci.edu/#/c/2165/1/asterixdb/asterix-transactions/src/ma

Change in asterixdb[master]: [NO ISSUE][COMP] Expression cleanup

2017-11-20 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][COMP] Expression cleanup .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2164 To unsubscribe, visit https://asteri

Change in asterixdb[master]: [ASTERIXDB-2164][HYR] Fix node assignment bug for unbounded ...

2017-11-17 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2164][HYR] Fix node assignment bug for unbounded task .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2148 To unsu

Change in asterixdb[master]: [ASTERIXDB-2165] Avoid OOM in QueryServiceServlet

2017-11-17 Thread Till Westmann (Code Review)
Till Westmann has submitted this change and it was merged. Change subject: [ASTERIXDB-2165] Avoid OOM in QueryServiceServlet .. [ASTERIXDB-2165] Avoid OOM in QueryServiceServlet - user model changes: no - storage format changes

Change in asterixdb[master]: [NO ISSUE] Check java version

2017-11-17 Thread Till Westmann (Code Review)
Till Westmann has submitted this change and it was merged. Change subject: [NO ISSUE] Check java version .. [NO ISSUE] Check java version - user model changes: no - storage format changes: no - interface changes: no Change-Id:

Change in asterixdb[master]: [ASTERIXDB-2165] Avoid OOM in QueryServiceServlet

2017-11-17 Thread Till Westmann (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, Michael Blow, Ian Maxon, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2150 to look at the new patch set (#3). Change subject: [ASTERIXDB-2165] Avoid OOM in QueryServiceServlet ...

Change in asterixdb[master]: [ASTERIXDB-2165] Avoid OOM in QueryServiceServlet

2017-11-17 Thread Till Westmann (Code Review)
Hello Anon. E. Moose #1000171, Jenkins, Michael Blow, Ian Maxon, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2150 to look at the new patch set (#2). Change subject: [ASTERIXDB-2165] Avoid OOM in QueryServiceServlet ...

Change in asterixdb[master]: [NO ISSUE][TX] Introduce Atomic Transactions

2017-11-16 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][TX] Introduce Atomic Transactions .. Patch Set 5: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/2156/4//COMMIT_MSG Commit Message: PS4, Line 13

Change in asterixdb[master]: [NO ISSUE][TX] Introduce Atomic Transactions

2017-11-16 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][TX] Introduce Atomic Transactions .. Patch Set 4: (9 comments) First round - mostly trivial. https://asterix-gerrit.ics.uci.edu/#/c/2156/4//COMMIT_M

Change in asterixdb[master]: [NO ISSUE] some plan printing improvements

2017-11-14 Thread Till Westmann (Code Review)
Till Westmann has submitted this change and it was merged. Change subject: [NO ISSUE] some plan printing improvements .. [NO ISSUE] some plan printing improvements Change-Id: Iea5a32a63e65636599dbc2a3674c1741d10e1d17 Reviewed-o

Change in asterixdb[master]: [ASTERIXDB-2165] Avoid OOM in QueryServiceServlet

2017-11-14 Thread Till Westmann (Code Review)
Till Westmann has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2150 Change subject: [ASTERIXDB-2165] Avoid OOM in QueryServiceServlet .. [ASTERIXDB-2165] Avoid OOM in QueryServiceServlet - user model c

Change in asterixdb[master]: [NO ISSUE][TX] Rename Asterix JobId to TxnId

2017-11-14 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][TX] Rename Asterix JobId to TxnId .. Patch Set 3: Code-Review+2 (2 comments) https://asterix-gerrit.ics.uci.edu/#/c/2149/3/asterixdb/asterix-metadata

Change in asterixdb[master]: [NO ISSUE][TX] Rename Asterix JobId to TxnId

2017-11-13 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][TX] Rename Asterix JobId to TxnId .. Patch Set 1: (10 comments) https://asterix-gerrit.ics.uci.edu/#/c/2149/1/asterixdb/asterix-app/src/main/java/org

Change in asterixdb[master]: [NO ISSUE] More tests for the Tracer

2017-11-13 Thread Till Westmann (Code Review)
Till Westmann has submitted this change and it was merged. Change subject: [NO ISSUE] More tests for the Tracer .. [NO ISSUE] More tests for the Tracer Change-Id: I9722c87232437c9ff3497ab9d1bcbef226094e0d Reviewed-on: https://a

Change in asterixdb[release-0.9.3-pre-rc]: [NO ISSUE][TX] Fix DatasetLock for Multiple Index Builds

2017-11-12 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][TX] Fix DatasetLock for Multiple Index Builds .. Patch Set 7: (2 comments) https://asterix-gerrit.ics.uci.edu/#/c/2144/7//COMMIT_MSG Commit Message:

Change in asterixdb[master]: [ASTERIXDB-1871][ASTERIXDB-2095] Stop Consumer Thread on Dea...

2017-11-10 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-1871][ASTERIXDB-2095] Stop Consumer Thread on Deallocate .. Patch Set 5: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2143 To

Change in asterixdb[master]: [ASTERIXDB-2115] Add Component Ids to LSM Indexes

2017-11-09 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2115] Add Component Ids to LSM Indexes .. Patch Set 50: Code-Review+1 (4 comments) Looks good to me, only cosmetic comments/questions. https://aster

Change in asterixdb[master]: [NO ISSUE][SQL] Fix token name typo in SQL++ parser

2017-11-09 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][SQL] Fix token name typo in SQL++ parser .. Patch Set 1: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2136 To unsubscribe, visi

Change in asterixdb[master]: [ASTERIXDB-2152][FUN][COMP] Enable specifying computation lo...

2017-11-09 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2152][FUN][COMP] Enable specifying computation location .. Patch Set 12: (2 comments) https://asterix-gerrit.ics.uci.edu/#/c/2114/12/asterixdb/aster

Change in asterixdb[master]: [ASTERIXDB-2158] Remove AsterixException (step 1)

2017-11-09 Thread Till Westmann (Code Review)
Till Westmann has submitted this change and it was merged. Change subject: [ASTERIXDB-2158] Remove AsterixException (step 1) .. [ASTERIXDB-2158] Remove AsterixException (step 1) - user model changes: no - storage format changes

Change in asterixdb-bad[master]: [NO ISSUE][FAIL] Replace MetadataException by AlgebricksExce...

2017-11-09 Thread Till Westmann (Code Review)
Till Westmann has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2139 Change subject: [NO ISSUE][FAIL] Replace MetadataException by AlgebricksException in APIs .. [NO ISSUE][FAIL] Replace MetadataExcepti

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

2017-11-08 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][COMP] Function references cleanup .. Patch Set 2: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/2130/2/asterixdb/asterix-lang-sqlpp/src/main/jav

Change in asterixdb[master]: [ASTERIXDB-2158] Remove AsterixException (step 1)

2017-11-07 Thread Till Westmann (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2135 to look at the new patch set (#4). Change subject: [ASTERIXDB-2158] Remove AsterixException (step 1) .. [ASTERIXDB-21

Change in asterixdb[master]: [ASTERIXDB-2158] Remove AsterixException (step 1)

2017-11-07 Thread Till Westmann (Code Review)
Till Westmann has uploaded a new patch set (#2). Change subject: [ASTERIXDB-2158] Remove AsterixException (step 1) .. [ASTERIXDB-2158] Remove AsterixException (step 1) - user model changes: no - storage format changes: no - inte

Change in asterixdb[master]: [ASTERIXDB-2158] Remove AsterixException (step 1)

2017-11-07 Thread Till Westmann (Code Review)
Till Westmann has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2135 Change subject: [ASTERIXDB-2158] Remove AsterixException (step 1) .. [ASTERIXDB-2158] Remove AsterixException (step 1) - user model c

Change in asterixdb[master]: [ASTERIXDB-2152][FUN][COMP] Enable specifying computation lo...

2017-11-06 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2152][FUN][COMP] Enable specifying computation location .. Patch Set 12: (3 comments) https://asterix-gerrit.ics.uci.edu/#/c/2114/12/asterixdb/aster

Change in asterixdb[master]: [ASTERIXDB-1911][HYR,RT,CLUS] Fixes and Improvements for Pre...

2017-11-06 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-1911][HYR,RT,CLUS] Fixes and Improvements for PreDistributed Jobs .. Patch Set 20: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/2045/14/hyrack

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

2017-11-06 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][RT][IDX] Simplify index.createAccessor() .. Patch Set 6: Code-Review+1 -- To view, visit https://asterix-gerrit.ics.uci.edu/2120 To unsubscribe, visi

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

2017-11-06 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][RT][IDX] Simplify index.createAccessor() .. Patch Set 6: (2 comments) https://asterix-gerrit.ics.uci.edu/#/c/2120/6/hyracks-fullstack/hyracks/hyracks

Change in asterixdb[master]: [NO ISSUE] Check java version

2017-11-03 Thread Till Westmann (Code Review)
Till Westmann has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2123 Change subject: [NO ISSUE] Check java version .. [NO ISSUE] Check java version - user model changes: no - storage format changes: no

Change in asterixdb[master]: [ASTERIXDB-1954][STO][RT] Add Index Drop Options

2017-11-03 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-1954][STO][RT] Add Index Drop Options .. Patch Set 2: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/2121/2/hyracks-fullstack/hyracks/hyracks-ipc

Change in asterixdb[master]: [ASTERIXDB-1911][HYR,RT,CLUS] Fixes and Improvements for Pre...

2017-11-02 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-1911][HYR,RT,CLUS] Fixes and Improvements for PreDistributed Jobs .. Patch Set 14: (5 comments) https://asterix-gerrit.ics.uci.edu/#/c/2045/14/aster

Change in asterixdb[master]: [NO ISSUE][STO] Fix memory leaks in storage

2017-11-02 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][STO] Fix memory leaks in storage .. Patch Set 6: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/2115/6/hyracks-fullstack/hyracks/hyracks-storage-

Change in asterixdb[master]: [ASTERIXDB-1952][TX][IDX]Filter logs pt.2

2017-11-02 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-1952][TX][IDX]Filter logs pt.2 .. Patch Set 19: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/1857/19/hyracks-fullstack/hyracks/hyracks-storage-

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

2017-11-01 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][RT][IDX] simplify createAccessor() .. Patch Set 1: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/2120/1/hyracks-fullstack/hyracks/hyracks-storag

Change in asterixdb[master]: [NO ISSUE] More tests for the Tracer

2017-11-01 Thread Till Westmann (Code Review)
Till Westmann has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2119 Change subject: [NO ISSUE] More tests for the Tracer .. [NO ISSUE] More tests for the Tracer Change-Id: I9722c87232437c9ff3497ab9d1bc

Change in asterixdb[master]: [NO ISSUE] some plan printing improvements

2017-10-31 Thread Till Westmann (Code Review)
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2117 to look at the new patch set (#3). Change subject: [NO ISSUE] some plan printing improvements .. [NO ISSUE] some plan

Change in asterixdb[master]: [NO ISSUE] some plan printing improvements

2017-10-31 Thread Till Westmann (Code Review)
Till Westmann has uploaded a new patch set (#2). Change subject: [NO ISSUE] some plan printing improvements .. [NO ISSUE] some plan printing improvements Change-Id: Iea5a32a63e65636599dbc2a3674c1741d10e1d17 --- M asterixdb/aste

Change in asterixdb[master]: [NO ISSUE] some plan printing improvements

2017-10-31 Thread Till Westmann (Code Review)
Till Westmann has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2117 Change subject: [NO ISSUE] some plan printing improvements .. [NO ISSUE] some plan printing improvements Change-Id: Iea5a32a63e656365

Change in asterixdb[master]: [UI] Allow logical plan to be viewed as JSON / formatted JSON

2017-10-31 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [UI] Allow logical plan to be viewed as JSON / formatted JSON .. Patch Set 29: Code-Review+2 (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/1885/28/hyracks-

Change in asterixdb[master]: [NO ISSUE][STO] Ensure no duplicate keys in component's meta...

2017-10-31 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][STO] Ensure no duplicate keys in component's metadata .. Patch Set 2: (2 comments) https://asterix-gerrit.ics.uci.edu/#/c/2115/2/hyracks-fullstack/hy

<    2   3   4   5   6   7   8   9   10   11   >