[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 25: Thanks for the review! Glad to get this fixed fina

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. IMPALA-3926: part 2: avoid setting LD_LIBRARY_PAT

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 24: Verified+1 -- To view, visit http://gerri

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 24: Build started: https://jenkins.impala.io/j

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 24: Build Successful https://jenkins.impala.i

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 24: (2 comments) http://gerrit.cloudera.org:8

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 24: Build started: https://jenkins.impala.io/j

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-06 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 24: Code-Review+2 -- To view, visit http://gerrit.clo

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-06 Thread Tim Armstrong (Code Review)
Hello Laszlo Gaal, Joe McDonnell, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14494 to look at the new patch set (#24). Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-06 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 23: This hit IMPALA-9729 -- To view, visit http://ge

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 23: Verified-1 Build failed: https://jenkins.i

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 22: Build Successful https://jenkins.impala.i

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 22: (1 comment) http://gerrit.cloudera.org:80

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 23: Code-Review+2 -- To view, visit http://ge

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-06 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 22: Code-Review+2 Fixed the test (mechanically applied

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 23: Build started: https://jenkins.impala.io/j

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-06 Thread Tim Armstrong (Code Review)
Hello Laszlo Gaal, Joe McDonnell, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14494 to look at the new patch set (#22). Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-06 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 21: Hit IMPALA-9728, and a shell test needs fixing -

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 21: Verified-1 Build failed: https://jenkins.i

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 21: Code-Review+2 -- To view, visit http://ge

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 21: Build started: https://jenkins.impala.io/j

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-05-05 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 20: Code-Review+2 This all looks good to me. Good to s

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-04-29 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 20: Ok, this also passed an exhaustive run on centOS 7

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-04-29 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 20: Verified+1 -- To view, visit http://gerri

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-04-28 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 20: Build Successful https://jenkins.impala.i

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-04-28 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 20: (1 comment) http://gerrit.cloudera.org:80

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-04-28 Thread Tim Armstrong (Code Review)
Hello Laszlo Gaal, Joe McDonnell, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14494 to look at the new patch set (#20). Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-04-28 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 20: Build started: https://jenkins.impala.io/j

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-04-28 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 18: Ran into issues with LD_LIBRARY_PATH from the test

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-04-27 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 16: (4 comments) This will cause a bit of annoyance w

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-04-17 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 16: Build Successful https://jenkins.impala.i

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-04-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14494 ) Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH .. Patch Set 16: I finally got back to this and fixed up the patch

[Impala-ASF-CR] IMPALA-3926: part 2: avoid setting LD LIBRARY PATH

2020-04-17 Thread Tim Armstrong (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14494 to look at the new patch set (#16). Change subject: IMPALA-3926: part 2: avoid setting LD_LIBRARY_PATH ..