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

2018-05-11 Thread Adam Holley (Code Review)
Adam Holley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10358 ) Change subject: IMPALA-6802 (part 3): Clean up authorization tests .. Patch Set 2: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-6907: Close stale connections to removed cluster members

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10327 ) Change subject: IMPALA-6907: Close stale connections to removed cluster members .. Patch Set 5: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-6907: Close stale connections to removed cluster members

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10327 ) Change subject: IMPALA-6907: Close stale connections to removed cluster members .. IMPALA-6907: Close stale

[Impala-ASF-CR](2.x) IMPALA-7010: don't run memory usage tests on non-HDFS

2018-05-11 Thread Tim Armstrong (Code Review)
Hello Impala Public Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10387 to review the following change. Change subject: IMPALA-7010: don't run memory usage tests on non-HDFS ..

[Impala-ASF-CR](2.x) IMPALA-7010: don't run memory usage tests on non-HDFS

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10387 ) Change subject: IMPALA-7010: don't run memory usage tests on non-HDFS .. Patch Set 1: Build started:

[Impala-ASF-CR] IMPALA-6957: calc thread resource requirement in planner

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10256 ) Change subject: IMPALA-6957: calc thread resource requirement in planner .. Patch Set 8: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-6957: calc thread resource requirement in planner

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10256 ) Change subject: IMPALA-6957: calc thread resource requirement in planner .. IMPALA-6957: calc thread resource

[Impala-ASF-CR] IMPALA-6221: [DOCS] Correct syntax for Kudu Partition clause

2018-05-11 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10353 ) Change subject: IMPALA-6221: [DOCS] Correct syntax for Kudu Partition clause .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-6221: [DOCS] Correct syntax for Kudu Partition clause

2018-05-11 Thread Alex Rodoni (Code Review)
Hello Thomas Tauber-Marshall, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10353 to look at the new patch set (#2). Change subject: IMPALA-6221: [DOCS] Correct syntax for Kudu Partition clause

[Impala-ASF-CR] IMPALA-7010: don't run memory usage tests on non-HDFS

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10370 ) Change subject: IMPALA-7010: don't run memory usage tests on non-HDFS .. Patch Set 5: This change did not cherrypick

[Impala-ASF-CR] IMPALA-6983: stress: don't write a null runtime profile

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10381 ) Change subject: IMPALA-6983: stress: don't write a null runtime profile .. Patch Set 1: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-7003: Deflake erasure coding data loading

2018-05-11 Thread Tianyi Wang (Code Review)
Tianyi Wang has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/10362 ) Change subject: IMPALA-7003: Deflake erasure coding data loading .. IMPALA-7003: Deflake erasure coding data loading Erasure coding data

[Impala-ASF-CR] IMPALA-5384, part 2: Simplify Coordinator locking and clarify state

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10158 ) Change subject: IMPALA-5384, part 2: Simplify Coordinator locking and clarify state .. IMPALA-5384, part 2: Simplify

[Impala-ASF-CR] IMPALA-5384, part 2: Simplify Coordinator locking and clarify state

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10158 ) Change subject: IMPALA-5384, part 2: Simplify Coordinator locking and clarify state .. Patch Set 21: Verified+1 -- To view,

[Impala-ASF-CR] IMPALA-6907: Close stale connections to removed cluster members

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10327 ) Change subject: IMPALA-6907: Close stale connections to removed cluster members .. Patch Set 5: Build started:

[Impala-ASF-CR] IMPALA-5216: Make admission control queuing async

2018-05-11 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/10060 ) Change subject: IMPALA-5216: Make admission control queuing async .. Patch Set 8: (1 comment)

[Impala-ASF-CR] IMPALA-5384, part 2: Simplify Coordinator locking and clarify state

2018-05-11 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10158 ) Change subject: IMPALA-5384, part 2: Simplify Coordinator locking and clarify state .. Patch Set 21: (1 comment)

[Impala-ASF-CR] IMPALA-6957: calc thread resource requirement in planner

2018-05-11 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/10256 ) Change subject: IMPALA-6957: calc thread resource requirement in planner .. Patch Set 4: (1 comment)

[Impala-ASF-CR] IMPALA-5216: Make admission control queuing async

2018-05-11 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/10060 ) Change subject: IMPALA-5216: Make admission control queuing async .. Patch Set 8: (1 comment)

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

2018-05-11 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/10358 ) Change subject: IMPALA-6802 (part 3): Clean up authorization tests .. Patch Set 2: (1 comment)

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

2018-05-11 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/10358 ) Change subject: IMPALA-6802 (part 3): Clean up authorization tests .. IMPALA-6802 (part 3): Clean up authorization tests The third part

[Impala-ASF-CR] IMPALA-5216: Make admission control queuing async

2018-05-11 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/10060 ) Change subject: IMPALA-5216: Make admission control queuing async .. Patch Set 7: (40 comments)

[Impala-ASF-CR] IMPALA-5216: Make admission control queuing async

2018-05-11 Thread Bikramjeet Vig (Code Review)
Hello Tim Armstrong, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10060 to look at the new patch set (#8). Change subject: IMPALA-5216: Make admission control queuing async ..

[Impala-ASF-CR] IMPALA-7010: don't run memory usage tests on non-HDFS

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10370 ) Change subject: IMPALA-7010: don't run memory usage tests on non-HDFS .. Patch Set 4: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7010: don't run memory usage tests on non-HDFS

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10370 ) Change subject: IMPALA-7010: don't run memory usage tests on non-HDFS .. IMPALA-7010: don't run memory usage tests on

[Impala-ASF-CR] IMPALA-6983: stress: don't write a null runtime profile

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10381 ) Change subject: IMPALA-6983: stress: don't write a null runtime profile .. Patch Set 1: Build started:

[Impala-ASF-CR] IMPALA-6983: stress: don't write a null runtime profile

2018-05-11 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/10381 ) Change subject: IMPALA-6983: stress: don't write a null runtime profile .. Patch Set 1: > I don't have permissions to give +2 >

[Impala-ASF-CR] IMPALA-6957: calc thread resource requirement in planner

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10256 ) Change subject: IMPALA-6957: calc thread resource requirement in planner .. Patch Set 8: Build started:

[Impala-ASF-CR] IMPALA-6957: calc thread resource requirement in planner

2018-05-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10256 ) Change subject: IMPALA-6957: calc thread resource requirement in planner .. Patch Set 8: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-6983: stress: don't write a null runtime profile

2018-05-11 Thread Nithya Janarthanan (Code Review)
Nithya Janarthanan has posted comments on this change. ( http://gerrit.cloudera.org:8080/10381 ) Change subject: IMPALA-6983: stress: don't write a null runtime profile .. Patch Set 1: Code-Review+1 I don't have permissions

[Impala-ASF-CR] IMPALA-6221: [DOCS] Correct syntax for Kudu Partition clause

2018-05-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/10353 ) Change subject: IMPALA-6221: [DOCS] Correct syntax for Kudu Partition clause .. Patch Set 1: (1 comment)

[Impala-ASF-CR](2.x) Ignore 3.x version update patch

2018-05-11 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10383 ) Change subject: Ignore 3.x version update patch .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/10383

[Impala-ASF-CR](2.x) Ignore 3.x version update patch

2018-05-11 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10383 ) Change subject: Ignore 3.x version update patch .. Ignore 3.x version update patch Change-Id:

[Impala-ASF-CR](2.x) Ignore 3.x version update patch

2018-05-11 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/10383 ) Change subject: Ignore 3.x version update patch .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR](2.x) Ignore 3.x version update patch

2018-05-11 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10383 Change subject: Ignore 3.x version update patch .. Ignore 3.x version update patch Change-Id:

[Impala-ASF-CR] IMPALA-5384, part 2: Simplify Coordinator locking and clarify state

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10158 ) Change subject: IMPALA-5384, part 2: Simplify Coordinator locking and clarify state .. Patch Set 21: Build started:

[Impala-ASF-CR] IMPALA-5384, part 2: Simplify Coordinator locking and clarify state

2018-05-11 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10158 ) Change subject: IMPALA-5384, part 2: Simplify Coordinator locking and clarify state .. Patch Set 21: Code-Review+2 Rebase -- To view,

[Impala-ASF-CR] IMPALA-5384, part 2: Simplify Coordinator locking and clarify state

2018-05-11 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10158 ) Change subject: IMPALA-5384, part 2: Simplify Coordinator locking and clarify state .. Patch Set 18: (1 comment)

[Impala-ASF-CR] IMPALA-5384, part 2: Simplify Coordinator locking and clarify state

2018-05-11 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10158 ) Change subject: IMPALA-5384, part 2: Simplify Coordinator locking and clarify state .. Patch Set 20: Code-Review+2 (4 comments)

[Impala-ASF-CR] IMPALA-3307: Add support for IANA time-zone db

2018-05-11 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/9986 ) Change subject: IMPALA-3307: Add support for IANA time-zone db .. Patch Set 5: (2 comments)

[Impala-ASF-CR] IMPALA-6957: calc thread resource requirement in planner

2018-05-11 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10256 ) Change subject: IMPALA-6957: calc thread resource requirement in planner .. Patch Set 7: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-6941: load more text scanner compression plugins

2018-05-11 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10165 ) Change subject: IMPALA-6941: load more text scanner compression plugins .. Patch Set 11: Thanks -- To view, visit

[Impala-ASF-CR] IMPALA-6941: load more text scanner compression plugins

2018-05-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10165 ) Change subject: IMPALA-6941: load more text scanner compression plugins .. Patch Set 11: I believe the test case below covers the

[Impala-ASF-CR] IMPALA-6907: Close stale connections to removed cluster members

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10327 ) Change subject: IMPALA-6907: Close stale connections to removed cluster members .. Patch Set 5: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-6957: calc thread resource requirement in planner

2018-05-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10256 ) Change subject: IMPALA-6957: calc thread resource requirement in planner .. Patch Set 6: (3 comments)

[Impala-ASF-CR] IMPALA-6957: calc thread resource requirement in planner

2018-05-11 Thread Tim Armstrong (Code Review)
Hello Bikramjeet Vig, Alex Behm, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10256 to look at the new patch set (#7). Change subject: IMPALA-6957: calc thread resource requirement in planner

[Impala-ASF-CR] IMPALA-6983: stress: don't write a null runtime profile

2018-05-11 Thread Michael Brown (Code Review)
Michael Brown has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10381 Change subject: IMPALA-6983: stress: don't write a null runtime profile .. IMPALA-6983: stress: don't write a null runtime profile

[Impala-ASF-CR] IMPALA-7010: don't run memory usage tests on non-HDFS

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10370 ) Change subject: IMPALA-7010: don't run memory usage tests on non-HDFS .. Patch Set 4: Build started:

[Impala-ASF-CR] IMPALA-7010: don't run memory usage tests on non-HDFS

2018-05-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10370 ) Change subject: IMPALA-7010: don't run memory usage tests on non-HDFS .. Patch Set 4: Code-Review+2 -- To view, visit

[Impala-ASF-CR] Update version to 3.1.0-SNAPSHOT

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10360 ) Change subject: Update version to 3.1.0-SNAPSHOT .. Patch Set 2: This change did not cherrypick successfully into branch 2.x.

[Impala-ASF-CR] Update version to 3.1.0-SNAPSHOT

2018-05-11 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/10360 ) Change subject: Update version to 3.1.0-SNAPSHOT .. Patch Set 2: Sorry for the spam; I'm testing something on the cherrypicker

[Impala-ASF-CR] Update version to 3.1.0-SNAPSHOT

2018-05-11 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/10360 ) Change subject: Update version to 3.1.0-SNAPSHOT .. Patch Set 2: test test -- To view, visit

[Impala-ASF-CR] IMPALA-7010: don't run memory usage tests on non-HDFS

2018-05-11 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10370 ) Change subject: IMPALA-7010: don't run memory usage tests on non-HDFS .. Patch Set 3: Code-Review+2 (1 comment)

[Impala-ASF-CR] Update version to 3.1.0-SNAPSHOT

2018-05-11 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/10360 ) Change subject: Update version to 3.1.0-SNAPSHOT .. Patch Set 2: --help -- To view, visit http://gerrit.cloudera.org:8080/10360

[Impala-ASF-CR] Update version to 3.1.0-SNAPSHOT

2018-05-11 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/10360 ) Change subject: Update version to 3.1.0-SNAPSHOT .. Patch Set 2: test -- To view, visit http://gerrit.cloudera.org:8080/10360 To

[Impala-ASF-CR] IMPALA-7010: don't run memory usage tests on non-HDFS

2018-05-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10370 ) Change subject: IMPALA-7010: don't run memory usage tests on non-HDFS .. Patch Set 2: (2 comments)

[Impala-ASF-CR] IMPALA-7010: don't run memory usage tests on non-HDFS

2018-05-11 Thread Tim Armstrong (Code Review)
Hello Sailesh Mukil, Impala Public Jenkins, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10370 to look at the new patch set (#3). Change subject: IMPALA-7010: don't run memory usage tests on non-HDFS

[Impala-ASF-CR] IMPALA-7010: don't run memory usage tests on non-HDFS

2018-05-11 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10370 ) Change subject: IMPALA-7010: don't run memory usage tests on non-HDFS .. Patch Set 2: (2 comments)

[Impala-ASF-CR] IMPALA-6907: Close stale connections to removed cluster members

2018-05-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10327 ) Change subject: IMPALA-6907: Close stale connections to removed cluster members .. Patch Set 5: Build started:

[Impala-ASF-CR] IMPALA-6907: Close stale connections to removed cluster members

2018-05-11 Thread Michael Ho (Code Review)
Hello Tianyi Wang, Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10327 to look at the new patch set (#5). Change subject: IMPALA-6907: Close stale connections to removed cluster members

[Impala-ASF-CR] IMPALA-6131: Track time of last statistics update in metadata

2018-05-11 Thread Csaba Ringhofer (Code Review)
Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/10116 ) Change subject: IMPALA-6131: Track time of last statistics update in metadata .. Patch Set 10: (8 comments)

[Impala-ASF-CR] IMPALA-5384, part 2: Simplify Coordinator locking and clarify state

2018-05-11 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10158 ) Change subject: IMPALA-5384, part 2: Simplify Coordinator locking and clarify state .. Patch Set 18: (1 comment)

[Impala-ASF-CR] IMPALA-6941: load more text scanner compression plugins

2018-05-11 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10165 ) Change subject: IMPALA-6941: load more text scanner compression plugins .. Patch Set 11: I wasn't worried so much about the load data

[Impala-ASF-CR] IMPALA-6957: calc thread resource requirement in planner

2018-05-11 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10256 ) Change subject: IMPALA-6957: calc thread resource requirement in planner .. Patch Set 6: (3 comments)

[Impala-ASF-CR] IMPALA-6941: load more text scanner compression plugins

2018-05-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10165 ) Change subject: IMPALA-6941: load more text scanner compression plugins .. Patch Set 11: Could you have a look at the fe changes,

[Impala-ASF-CR] IMPALA-6941: load more text scanner compression plugins

2018-05-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10165 ) Change subject: IMPALA-6941: load more text scanner compression plugins .. Patch Set 11: There was no existing test coverage for the

[Impala-ASF-CR] IMPALA-5384, part 2: Simplify Coordinator locking and clarify state

2018-05-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10158 ) Change subject: IMPALA-5384, part 2: Simplify Coordinator locking and clarify state .. Patch Set 18: (1 comment)

[Impala-ASF-CR] IMPALA-6941: load more text scanner compression plugins

2018-05-11 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10165 ) Change subject: IMPALA-6941: load more text scanner compression plugins .. Patch Set 11: Code-Review+1 That seems okay to me, though

[Impala-ASF-CR] IMPALA-5384, part 2: Simplify Coordinator locking and clarify state

2018-05-11 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10158 ) Change subject: IMPALA-5384, part 2: Simplify Coordinator locking and clarify state .. Patch Set 20: Code-Review+1 Carry +1s -- To

[Impala-ASF-CR] IMPALA-6131: Track time of last statistics update in metadata

2018-05-11 Thread Csaba Ringhofer (Code Review)
Hello Lars Volker, Zoltan Borok-Nagy, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10116 to look at the new patch set (#11). Change subject: IMPALA-6131: Track time of last statistics update in metadata

[Impala-ASF-CR] IMPALA-5384, part 2: Simplify Coordinator locking and clarify state

2018-05-11 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10158 ) Change subject: IMPALA-5384, part 2: Simplify Coordinator locking and clarify state .. Patch Set 20: (12 comments) Thanks for taking a

[Impala-ASF-CR] IMPALA-5384, part 2: Simplify Coordinator locking and clarify state

2018-05-11 Thread Dan Hecht (Code Review)
Hello Michael Ho, Sailesh Mukil, Tim Armstrong, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10158 to look at the new patch set (#20). Change subject: IMPALA-5384, part 2: Simplify Coordinator locking and clarify state

[Impala-ASF-CR] IMPALA-6997: Avoid redundant dumping in SetMemLimitExceeded()

2018-05-11 Thread Zoram Thanga (Code Review)
Zoram Thanga has posted comments on this change. ( http://gerrit.cloudera.org:8080/10364 ) Change subject: IMPALA-6997: Avoid redundant dumping in SetMemLimitExceeded() .. Patch Set 1: (1 comment)

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

2018-05-11 Thread Adam Holley (Code Review)
Adam Holley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10358 ) Change subject: IMPALA-6802 (part 3): Clean up authorization tests .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-3307: Add support for IANA time-zone db

2018-05-11 Thread Attila Jeges (Code Review)
Hello Gabor Kaszab, Zoltan Borok-Nagy, Csaba Ringhofer, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9986 to look at the new patch set (#6). Change subject: IMPALA-3307: Add support for IANA time-zone db

[Impala-ASF-CR] IMPALA-3307: Add support for IANA time-zone db

2018-05-11 Thread Csaba Ringhofer (Code Review)
Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/9986 ) Change subject: IMPALA-3307: Add support for IANA time-zone db .. Patch Set 5: (1 comment)

[Impala-ASF-CR] IMPALA-3307: Add support for IANA time-zone db

2018-05-11 Thread Attila Jeges (Code Review)
Attila Jeges has posted comments on this change. ( http://gerrit.cloudera.org:8080/9986 ) Change subject: IMPALA-3307: Add support for IANA time-zone db .. Patch Set 4: (18 comments)