[Impala-ASF-CR] Give each config change in bootstrap system.sh its own line

2019-04-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12900 ) Change subject: Give each config change in bootstrap_system.sh its own line .. Patch Set 4: Build Successful

[Impala-ASF-CR] Give each config change in bootstrap system.sh its own line

2019-04-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12900 ) Change subject: Give each config change in bootstrap_system.sh its own line .. Patch Set 4: (2 comments)

[Impala-ASF-CR] Give each config change in bootstrap system.sh its own line

2019-04-01 Thread Jim Apple (Code Review)
Hello Lars Volker, Joe McDonnell, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12900 to look at the new patch set (#4). Change subject: Give each config change in bootstrap_system.sh its own line

[Impala-ASF-CR] Give each config change in bootstrap system.sh its own line

2019-04-01 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. ( http://gerrit.cloudera.org:8080/12900 ) Change subject: Give each config change in bootstrap_system.sh its own line .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-8143: Enhance DoRpcWithRetry().

2019-04-01 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/12672 ) Change subject: IMPALA-8143: Enhance DoRpcWithRetry(). .. Patch Set 7: (5 comments)

[Impala-ASF-CR] [DOCS] Use -- to denote startup flags for consistency

2019-04-01 Thread Alex Rodoni (Code Review)
Alex Rodoni has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12908 ) Change subject: [DOCS] Use -- to denote startup flags for consistency .. [DOCS] Use -- to denote startup flags for consistency

[Impala-ASF-CR] [DOCS] Use -- to denote startup flags for consistency

2019-04-01 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/12908 ) Change subject: [DOCS] Use -- to denote startup flags for consistency .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] [DOCS] Use -- to denote startup flags for consistency

2019-04-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12908 ) Change subject: [DOCS] Use -- to denote startup flags for consistency .. Patch Set 2: Verified+1 Build Successful

[Impala-ASF-CR] [DOCS] Use -- to denote startup flags for consistency

2019-04-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12908 ) Change subject: [DOCS] Use -- to denote startup flags for consistency .. Patch Set 1: Verified+1 Build Successful

[Impala-ASF-CR] [DOCS] Use -- to denote startup flags for consistency

2019-04-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12908 ) Change subject: [DOCS] Use -- to denote startup flags for consistency .. Patch Set 2: Build Started

[Impala-ASF-CR] [DOCS] Use -- to denote startup flags for consistency

2019-04-01 Thread Alex Rodoni (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12908 to look at the new patch set (#2). Change subject: [DOCS] Use -- to denote startup flags for consistency

[Impala-ASF-CR] IMPALA-2990: timeout unresponsive queries in coordinator

2019-04-01 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/12299 ) Change subject: IMPALA-2990: timeout unresponsive queries in coordinator .. Patch Set 7: (6 comments)

[Impala-ASF-CR] [DOCS] Use -- to denote startup flags for consistency

2019-04-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12908 ) Change subject: [DOCS] Use -- to denote startup flags for consistency .. Patch Set 1: Build Started

[Impala-ASF-CR] [DOCS] Use -- to denote startup flags for consistency

2019-04-01 Thread Alex Rodoni (Code Review)
Alex Rodoni has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12908 Change subject: [DOCS] Use -- to denote startup flags for consistency .. [DOCS] Use -- to denote startup flags for consistency

[Impala-ASF-CR] Fix stress test tracking of past metrics values

2019-04-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12903 ) Change subject: Fix stress test tracking of past metrics values .. Fix stress test tracking of past metrics values

[Impala-ASF-CR] Fix stress test tracking of past metrics values

2019-04-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12903 ) Change subject: Fix stress test tracking of past metrics values .. Patch Set 1: Verified+1 -- To view, visit

[Impala-ASF-CR] Improve error handling for validation of unified backend executable

2019-04-01 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/12895 ) Change subject: Improve error handling for validation of unified backend executable .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7971: Add support for insert events in event processor.

2019-04-01 Thread Bharath Krishna (Code Review)
Bharath Krishna has posted comments on this change. ( http://gerrit.cloudera.org:8080/12889 ) Change subject: IMPALA-7971: Add support for insert events in event processor. .. Patch Set 4: (1 comment)

[Impala-ASF-CR] Revert "Bump toolchain version to 8430101a-b65f-4e4d-921e-767771649ae4-713afc77a8"

2019-04-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12905 ) Change subject: Revert "Bump toolchain version to 8430101a-b65f-4e4d-921e-767771649ae4-713afc77a8" .. Patch Set 1: Build

[Impala-ASF-CR] Fix stress test tracking of past metrics values

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12903 ) Change subject: Fix stress test tracking of past metrics values .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] Fix stress test tracking of past metrics values

2019-04-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12903 ) Change subject: Fix stress test tracking of past metrics values .. Patch Set 1: Build started:

[Impala-ASF-CR] Revert "Bump toolchain version to 8430101a-b65f-4e4d-921e-767771649ae4-713afc77a8"

2019-04-01 Thread Thomas Marshall (Code Review)
Thomas Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12905 Change subject: Revert "Bump toolchain version to 8430101a-b65f-4e4d-921e-767771649ae4-713afc77a8" .. Revert "Bump toolchain version

[native-toolchain-CR] Include python-dev in all build images

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12882 ) Change subject: Include python-dev in all build images .. Include python-dev in all build images Thrift doesn't fail the

[Impala-ASF-CR] Fix stress test tracking of past metrics values

2019-04-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12903 ) Change subject: Fix stress test tracking of past metrics values .. Patch Set 1: Build Successful

[Impala-ASF-CR] Ensure ssh config change is on its own line

2019-04-01 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/12900 ) Change subject: Ensure ssh config change is on its own line .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-8309: add user authorization provider flag

2019-04-01 Thread Austin Nobis (Code Review)
Austin Nobis has posted comments on this change. ( http://gerrit.cloudera.org:8080/12901 ) Change subject: IMPALA-8309: add user authorization_provider flag .. Patch Set 2: (13 comments) You also need to add E2E tests in the

[Impala-ASF-CR] Fix stress test tracking of past metrics values

2019-04-01 Thread Thomas Marshall (Code Review)
Thomas Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12903 Change subject: Fix stress test tracking of past metrics values .. Fix stress test tracking of past metrics values The stress test

[native-toolchain-CR] Parallelize image building process

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12864 ) Change subject: Parallelize image building process .. Parallelize image building process Images can now be built in parallel

[native-toolchain-CR] Fix debian:7 and debian:8 image builds

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12863 ) Change subject: Fix debian:7 and debian:8 image builds .. Fix debian:7 and debian:8 image builds Wheezy and Jessie were moved

[native-toolchain-CR] Work around flatbuffers flaky build

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12886 ) Change subject: Work around flatbuffers flaky build .. Patch Set 4: Verified+1 Code-Review+2 -- To view, visit

[native-toolchain-CR] Parallelize image building process

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12864 ) Change subject: Parallelize image building process .. Patch Set 4: Verified+1 Code-Review+2 -- To view, visit

[native-toolchain-CR] Work around flatbuffers flaky build

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12886 ) Change subject: Work around flatbuffers flaky build .. Work around flatbuffers flaky build The flatbuffers build ocasionally

[native-toolchain-CR] Include python-dev in all build images

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12882 ) Change subject: Include python-dev in all build images .. Patch Set 4: Verified+1 Code-Review+2 -- To view, visit

[native-toolchain-CR] Fix debian:7 and debian:8 image builds

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12863 ) Change subject: Fix debian:7 and debian:8 image builds .. Patch Set 4: Verified+1 Code-Review+2 -- To view, visit

[native-toolchain-CR] Prepend docker image to both stderr and stdout

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12861 ) Change subject: Prepend docker image to both stderr and stdout .. Prepend docker image to both stderr and stdout Sometimes,

[native-toolchain-CR] Install newer nss version in centos 6.6

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12862 ) Change subject: Install newer nss version in centos 6.6 .. Install newer nss version in centos 6.6 Centos 6.6 ships with an

[native-toolchain-CR] Work around flatbuffers flaky build

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12886 ) Change subject: Work around flatbuffers flaky build .. Patch Set 3: Verified+1 Code-Review+2 -- To view, visit

[native-toolchain-CR] Install newer nss version in centos 6.6

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12862 ) Change subject: Install newer nss version in centos 6.6 .. Patch Set 4: Code-Review+2 -- To view, visit

[native-toolchain-CR] Parallelize image building process

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12864 ) Change subject: Parallelize image building process .. Patch Set 3: Verified+1 Code-Review+2 (1 comment)

[Impala-ASF-CR] Improve error handling for validation of unified backend executable

2019-04-01 Thread Andrew Sherman (Code Review)
Andrew Sherman has posted comments on this change. ( http://gerrit.cloudera.org:8080/12895 ) Change subject: Improve error handling for validation of unified backend executable .. Patch Set 1: Code-Review+1 LGTM -- To

[native-toolchain-CR] Fix debian:7 and debian:8 image builds

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12863 ) Change subject: Fix debian:7 and debian:8 image builds .. Patch Set 3: Verified+1 Code-Review+2 -- To view, visit

[native-toolchain-CR] Prepend docker image to both stderr and stdout

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12861 ) Change subject: Prepend docker image to both stderr and stdout .. Patch Set 2: Code-Review+2 -- To view, visit

[native-toolchain-CR] Prepend docker image to both stderr and stdout

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12861 ) Change subject: Prepend docker image to both stderr and stdout .. Patch Set 2: Verified+1 -- To view, visit

[native-toolchain-CR] Include python-dev in all build images

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12882 ) Change subject: Include python-dev in all build images .. Patch Set 3: Hector sent me a pointer to the completed build. -- To view,

[native-toolchain-CR] Include python-dev in all build images

2019-04-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12882 ) Change subject: Include python-dev in all build images .. Patch Set 3: Verified+1 -- To view, visit

[Impala-ASF-CR](2.x) IMPALA-7211: Fix the between predicate for decimals

2019-04-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12902 ) Change subject: IMPALA-7211: Fix the between predicate for decimals .. Patch Set 3: Build Successful

[Impala-ASF-CR](2.x) IMPALA-7211: Fix the between predicate for decimals

2019-04-01 Thread Quanlong Huang (Code Review)
Hello Vuk Ercegovac, Impala Public Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/12902 to review the following change. Change subject: IMPALA-7211: Fix the between predicate for decimals

[Impala-ASF-CR](2.x) IMPALA-7211: Fix the between predicate for decimals

2019-04-01 Thread Quanlong Huang (Code Review)
Quanlong Huang has posted comments on this change. ( http://gerrit.cloudera.org:8080/12902 ) Change subject: IMPALA-7211: Fix the between predicate for decimals .. Patch Set 3: Cherry-pick conflicts:

[Impala-ASF-CR] Make Docker-based tests available on more hosting OS platforms

2019-04-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12892 ) Change subject: Make Docker-based tests available on more hosting OS platforms .. Patch Set 3: Build Successful

[Impala-ASF-CR] Make Docker-based tests available on more hosting OS platforms

2019-04-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12892 ) Change subject: Make Docker-based tests available on more hosting OS platforms .. Patch Set 2: Build Successful

[Impala-ASF-CR] Make Docker-based tests available on more hosting OS platforms

2019-04-01 Thread Laszlo Gaal (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12892 to look at the new patch set (#2). Change subject: Make Docker-based tests available on more hosting OS platforms

[Impala-ASF-CR] Make Docker-based tests available on more hosting OS platforms

2019-04-01 Thread Laszlo Gaal (Code Review)
Laszlo Gaal has posted comments on this change. ( http://gerrit.cloudera.org:8080/12892 ) Change subject: Make Docker-based tests available on more hosting OS platforms .. Patch Set 3: (3 comments) Fixed flake8 style

[Impala-ASF-CR] Make Docker-based tests available on more hosting OS platforms

2019-04-01 Thread Laszlo Gaal (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12892 to look at the new patch set (#3). Change subject: Make Docker-based tests available on more hosting OS platforms

[Impala-ASF-CR] Make Docker-based tests available on more hosting OS platforms

2019-04-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12892 ) Change subject: Make Docker-based tests available on more hosting OS platforms .. Patch Set 2: (2 comments)

[Impala-ASF-CR] IMPALA-8309: add user authorization provider flag

2019-04-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12901 ) Change subject: IMPALA-8309: add user authorization_provider flag .. Patch Set 2: Build Successful

[Impala-ASF-CR] IMPALA-8309: add user authorization provider flag

2019-04-01 Thread radford nguyen (Code Review)
radford nguyen has posted comments on this change. ( http://gerrit.cloudera.org:8080/12901 ) Change subject: IMPALA-8309: add user authorization_provider flag .. Patch Set 2: (10 comments) > Patch Set 1: > > (8 comments)

[Impala-ASF-CR] IMPALA-8309: add user authorization provider flag

2019-04-01 Thread radford nguyen (Code Review)
Hello Austin Nobis, Fredy Wijaya, Todd Lipcon, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12901 to look at the new patch set (#2). Change subject: IMPALA-8309: add user authorization_provider flag

[Impala-ASF-CR] IMPALA-8309: add user authorization provider flag

2019-04-01 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12901 ) Change subject: IMPALA-8309: add user authorization_provider flag .. Patch Set 1: Build Successful