[Impala-ASF-CR] WIP IMPALA-7178: Add the possibility to reduce logging for common data errors

Zoltan Borok-Nagy has posted comments on this change. ( http://gerrit.cloudera.org:8080/10793 ) Change subject: WIP IMPALA-7178: Add the possibility to reduce logging for common data errors .. Patch Set 1: (4 comments) I

[Impala-ASF-CR] [DOCS] Update to the workaround for IMPALA-3316

Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/10800 ) Change subject: [DOCS] Update to the workaround for IMPALA-3316 .. Patch Set 1: (1 comment)

[Impala-ASF-CR] [DOCS] Update to the workaround for IMPALA-3316

Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10800 ) Change subject: [DOCS] Update to the workaround for IMPALA-3316 .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-6802 (part 5): Clean up authorization tests

Adam Holley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10799 ) Change subject: IMPALA-6802 (part 5): Clean up authorization tests .. Patch Set 2: Code-Review+1 (1 comment) Just 1 nit.

[Impala-ASF-CR] IMPALA-6802 (part 5): Clean up authorization tests

Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/10799 ) Change subject: IMPALA-6802 (part 5): Clean up authorization tests .. Patch Set 3: Code-Review+1 (1 comment) Carry +1

[Impala-ASF-CR] IMPALA-6802 (part 5): Clean up authorization tests

Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/10799 ) Change subject: IMPALA-6802 (part 5): Clean up authorization tests .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7207: make Coordinator::exec state an atomic enum

Dan Hecht has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10811 Change subject: IMPALA-7207: make Coordinator::exec_state_ an atomic enum .. IMPALA-7207: make Coordinator::exec_state_ an atomic enum

[Impala-ASF-CR] [DOCS] Update to the workaround for IMPALA-3316

Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/10800 ) Change subject: [DOCS] Update to the workaround for IMPALA-3316 .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7207: make Coordinator::exec state an atomic enum

Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10811 ) Change subject: IMPALA-7207: make Coordinator::exec_state_ an atomic enum .. Patch Set 2: (2 comments)

[Impala-ASF-CR] WIP IMPALA-7178: Add the possibility to reduce logging for common data errors

Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10793 ) Change subject: WIP IMPALA-7178: Add the possibility to reduce logging for common data errors .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-7207: make Coordinator::exec state an atomic enum

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10811 ) Change subject: IMPALA-7207: make Coordinator::exec_state_ an atomic enum .. Patch Set 3: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7163: Implement a state machine for the QueryState class

Sailesh Mukil has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/10813 ) Change subject: IMPALA-7163: Implement a state machine for the QueryState class .. IMPALA-7163: Implement a state machine for the

[Impala-ASF-CR] IMPALA-7207: make Coordinator::exec state an atomic enum

Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10811 ) Change subject: IMPALA-7207: make Coordinator::exec_state_ an atomic enum .. IMPALA-7207: make

[Impala-ASF-CR] IMPALA-7163: Implement a state machine for the QueryState class

Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10813 ) Change subject: IMPALA-7163: Implement a state machine for the QueryState class .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-6816: minimise calls to GetMinSubscriberTopicVersion()

Tianyi Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10705 ) Change subject: IMPALA-6816: minimise calls to GetMinSubscriberTopicVersion() .. Patch Set 4: (1 comment)

[Impala-ASF-CR] WIP IMPALA-7178: Add the possibility to reduce logging for common data errors

Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10793 ) Change subject: WIP IMPALA-7178: Add the possibility to reduce logging for common data errors .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-6802 (part 5): Clean up authorization tests

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10799 ) Change subject: IMPALA-6802 (part 5): Clean up authorization tests .. Patch Set 4: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-6425: reduce MemPool max chunk size

Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10789 ) Change subject: IMPALA-6425: reduce MemPool max chunk size .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-7210: global debug actions should be case insensitive

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10814 ) Change subject: IMPALA-7210: global debug actions should be case insensitive .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-7210: global debug actions should be case insensitive

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10814 ) Change subject: IMPALA-7210: global debug actions should be case insensitive .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-3040: Remove cache directive before dropping a table

Tianyi Wang has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/10792 ) Change subject: IMPALA-3040: Remove cache directive before dropping a table .. IMPALA-3040: Remove cache directive before dropping a table

[Impala-ASF-CR] IMPALA-6802 (part 5): Clean up authorization tests

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10799 ) Change subject: IMPALA-6802 (part 5): Clean up authorization tests .. Patch Set 4: Build started:

[Impala-ASF-CR] IMPALA-4848: Add WIDTH BUCKET() function

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/6023 ) Change subject: IMPALA-4848: Add WIDTH_BUCKET() function .. Patch Set 17: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-110 (part 2): Refactor PartitionedAggregationNode

Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10394 ) Change subject: IMPALA-110 (part 2): Refactor PartitionedAggregationNode .. Patch Set 4: Code-Review+1 (3 comments) This all seems

[Impala-ASF-CR] Revert "IMPALA-5893: Remove old kinit code for Impala 3"

Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10563 ) Change subject: Revert "IMPALA-5893: Remove old kinit code for Impala 3" .. Patch Set 1: Do we have a plan for addressing IMPALA-7072 and

[Impala-ASF-CR] IMPALA-7140 (part 5): support fetching file info for FS tables

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10749 ) Change subject: IMPALA-7140 (part 5): support fetching file info for FS tables .. Patch Set 4: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-6425: reduce MemPool max chunk size

Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10789 ) Change subject: IMPALA-6425: reduce MemPool max chunk size .. Patch Set 1: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-7149: Disable some tests in the EC build

Tianyi Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10804 ) Change subject: IMPALA-7149: Disable some tests in the EC build .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7205: Respond to ReportExecStatus() RPC with CANCELLED if query execution has terminated

Dan Hecht has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10815 Change subject: IMPALA-7205: Respond to ReportExecStatus() RPC with CANCELLED if query execution has terminated .. IMPALA-7205: Respond to

[Impala-ASF-CR] IMPALA-6425: reduce MemPool max chunk size

Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10789 ) Change subject: IMPALA-6425: reduce MemPool max chunk size .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-7149: Disable some tests in the EC build

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10804 ) Change subject: IMPALA-7149: Disable some tests in the EC build .. Patch Set 2: Build started:

[Impala-ASF-CR] WIP IMPALA-7178: Add the possibility to reduce logging for common data errors

Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10793 ) Change subject: WIP IMPALA-7178: Add the possibility to reduce logging for common data errors .. Patch Set 1: (2 comments)

[Impala-ASF-CR] IMPALA-5981: [DOCS] Documented SET=""

Alex Rodoni has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10816 Change subject: IMPALA-5981: [DOCS] Documented SET="" .. IMPALA-5981: [DOCS] Documented SET="" Also, refactored the Impala SET doc and

[Impala-ASF-CR] IMPALA-5981: [DOCS] Documented SET=""

Alex Rodoni has removed Jim Apple from this change. ( http://gerrit.cloudera.org:8080/10816 ) Change subject: IMPALA-5981: [DOCS] Documented SET="" .. Removed reviewer Jim Apple. -- To view, visit

[Impala-ASF-CR] IMPALA-7140 (part 5): support fetching file info for FS tables

Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/10749 ) Change subject: IMPALA-7140 (part 5): support fetching file info for FS tables .. Patch Set 4: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7149: Disable some tests in the EC build

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10804 ) Change subject: IMPALA-7149: Disable some tests in the EC build .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-6425: reduce MemPool max chunk size

Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/10789 ) Change subject: IMPALA-6425: reduce MemPool max chunk size .. Patch Set 1: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7140 (part 5): support fetching file info for FS tables

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10749 ) Change subject: IMPALA-7140 (part 5): support fetching file info for FS tables .. Patch Set 4: Build started:

[Impala-ASF-CR] IMPALA-7207: make Coordinator::exec state an atomic enum

Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10811 ) Change subject: IMPALA-7207: make Coordinator::exec_state_ an atomic enum .. Patch Set 2: (2 comments)

[Impala-ASF-CR] Bump toolchain version, include libunwind

Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10755 ) Change subject: Bump toolchain version, include libunwind .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-3040: Remove cache directive before dropping a table

Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/10792 ) Change subject: IMPALA-3040: Remove cache directive before dropping a table .. Patch Set 1: (2 comments)

[Impala-ASF-CR] IMPALA-6305: Allow column definitions in ALTER VIEW

Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/10720 ) Change subject: IMPALA-6305: Allow column definitions in ALTER VIEW .. Patch Set 6: (2 comments)

[Impala-ASF-CR] IMPALA-7210: global debug actions should be case insensitive

Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10814 ) Change subject: IMPALA-7210: global debug actions should be case insensitive .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] [DOCS] Update to the workaround for IMPALA-3316

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10800 ) Change subject: [DOCS] Update to the workaround for IMPALA-3316 .. Patch Set 2: Verified+1 -- To view, visit

[Impala-ASF-CR] [DOCS] Update to the workaround for IMPALA-3316

Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10800 ) Change subject: [DOCS] Update to the workaround for IMPALA-3316 .. [DOCS] Update to the workaround for IMPALA-3316

[Impala-ASF-CR] IMPALA-7207: make Coordinator::exec state an atomic enum

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10811 ) Change subject: IMPALA-7207: make Coordinator::exec_state_ an atomic enum .. Patch Set 3: Build started:

[Impala-ASF-CR] IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans

Pooja Nilangekar has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/10704 ) Change subject: IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans .. IMPALA-6625: Skip computing parquet conjuncts

[Impala-ASF-CR] IMPALA-7210: global debug actions should be case insensitive

Dan Hecht has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10814 Change subject: IMPALA-7210: global debug actions should be case insensitive .. IMPALA-7210: global debug actions should be case

[Impala-ASF-CR] IMPALA-6305: Allow column definitions in ALTER VIEW

Pooja Nilangekar has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/10720 ) Change subject: IMPALA-6305: Allow column definitions in ALTER VIEW .. IMPALA-6305: Allow column definitions in ALTER VIEW This

[Impala-ASF-CR] IMPALA-7207: make Coordinator::exec state an atomic enum

Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10811 ) Change subject: IMPALA-7207: make Coordinator::exec_state_ an atomic enum .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-6305: Allow column definitions in ALTER VIEW

Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/10720 ) Change subject: IMPALA-6305: Allow column definitions in ALTER VIEW .. Patch Set 4: (9 comments) LGTM overall, I just have couple

[Impala-ASF-CR] IMPALA-7207: make Coordinator::exec state an atomic enum

Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10811 ) Change subject: IMPALA-7207: make Coordinator::exec_state_ an atomic enum .. Patch Set 2: (2 comments)

[Impala-ASF-CR] IMPALA-7207: make Coordinator::exec state an atomic enum

Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10811 ) Change subject: IMPALA-7207: make Coordinator::exec_state_ an atomic enum .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-4848: Add WIDTH BUCKET() function

Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/6023 ) Change subject: IMPALA-4848: Add WIDTH_BUCKET() function .. Patch Set 16: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-4848: Add WIDTH BUCKET() function

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/6023 ) Change subject: IMPALA-4848: Add WIDTH_BUCKET() function .. Patch Set 17: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-4848: Add WIDTH BUCKET() function

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/6023 ) Change subject: IMPALA-4848: Add WIDTH_BUCKET() function .. Patch Set 17: Build started:

[Impala-ASF-CR] IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans

Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10704 ) Change subject: IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans .. Patch Set 3: I don't think we need to

[Impala-ASF-CR] IMPALA-6305: Allow column definitions in ALTER VIEW

Pooja Nilangekar has posted comments on this change. ( http://gerrit.cloudera.org:8080/10720 ) Change subject: IMPALA-6305: Allow column definitions in ALTER VIEW .. Patch Set 4: (18 comments) Hey Vuk, Fredy, I am sorry I

[Impala-ASF-CR] IMPALA-7207: make Coordinator::exec state an atomic enum

Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10811 ) Change subject: IMPALA-7207: make Coordinator::exec_state_ an atomic enum .. Patch Set 2: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-6305: Allow column definitions in ALTER VIEW

Pooja Nilangekar has uploaded a new patch set (#6). ( http://gerrit.cloudera.org:8080/10720 ) Change subject: IMPALA-6305: Allow column definitions in ALTER VIEW .. IMPALA-6305: Allow column definitions in ALTER VIEW This

[Impala-ASF-CR] IMPALA-6305: Allow column definitions in ALTER VIEW

Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/10720 ) Change subject: IMPALA-6305: Allow column definitions in ALTER VIEW .. Patch Set 6: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-7207: make Coordinator::exec state an atomic enum

Hello Sailesh Mukil, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10811 to look at the new patch set (#3). Change subject: IMPALA-7207: make Coordinator::exec_state_ an atomic enum

[Impala-ASF-CR] IMPALA-7207: make Coordinator::exec state an atomic enum

Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10811 ) Change subject: IMPALA-7207: make Coordinator::exec_state_ an atomic enum .. Patch Set 3: (2 comments)

[Impala-ASF-CR] IMPALA-7207: make Coordinator::exec state an atomic enum

Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10811 ) Change subject: IMPALA-7207: make Coordinator::exec_state_ an atomic enum .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-7207: make Coordinator::exec state an atomic enum

Dan Hecht has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/10811 ) Change subject: IMPALA-7207: make Coordinator::exec_state_ an atomic enum .. IMPALA-7207: make Coordinator::exec_state_ an atomic enum That

[Impala-ASF-CR] IMPALA-6305: Allow column definitions in ALTER VIEW

Pooja Nilangekar has posted comments on this change. ( http://gerrit.cloudera.org:8080/10720 ) Change subject: IMPALA-6305: Allow column definitions in ALTER VIEW .. Patch Set 5: (8 comments)

[Impala-ASF-CR] IMPALA-6305: Allow column definitions in ALTER VIEW

Pooja Nilangekar has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/10720 ) Change subject: IMPALA-6305: Allow column definitions in ALTER VIEW .. IMPALA-6305: Allow column definitions in ALTER VIEW This

[Impala-ASF-CR] IMPALA-7149: Disable some tests in the EC build

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10804 ) Change subject: IMPALA-7149: Disable some tests in the EC build .. Patch Set 2: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-6802 (part 5): Clean up authorization tests

Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10799 ) Change subject: IMPALA-6802 (part 5): Clean up authorization tests .. IMPALA-6802 (part 5): Clean up authorization

[Impala-ASF-CR] IMPALA-6802 (part 5): Clean up authorization tests

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10799 ) Change subject: IMPALA-6802 (part 5): Clean up authorization tests .. Patch Set 4: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7140 (part 5): support fetching file info for FS tables

Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10749 ) Change subject: IMPALA-7140 (part 5): support fetching file info for FS tables .. IMPALA-7140 (part 5): support fetching file

[Impala-ASF-CR] IMPALA-7121: Clean up partitionIds from HdfsTable

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10654 ) Change subject: IMPALA-7121: Clean up partitionIds_ from HdfsTable .. Patch Set 7: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7121: Clean up partitionIds from HdfsTable

Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10654 ) Change subject: IMPALA-7121: Clean up partitionIds_ from HdfsTable .. IMPALA-7121: Clean up partitionIds_ from

[Impala-ASF-CR] IMPALA-7121: Clean up partitionIds from HdfsTable

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10654 ) Change subject: IMPALA-7121: Clean up partitionIds_ from HdfsTable .. Patch Set 7: Build started:

[Impala-ASF-CR] IMPALA-7121: Clean up partitionIds from HdfsTable

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10654 ) Change subject: IMPALA-7121: Clean up partitionIds_ from HdfsTable .. Patch Set 7: Code-Review+2 -- To view, visit