[Impala-ASF-CR] Don't assume that AUX exists just because a shell variable is set

2016-10-03 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Don't assume that AUX exists just because a shell variable is set .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4563/2/tests/run-custom-cluster-t

[Impala-ASF-CR] Don't assume that AUX exists just because a shell variable is set

2016-10-03 Thread Jim Apple (Code Review)
Hello Michael Brown, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4563 to look at the new patch set (#3). Change subject: Don't assume that AUX exists just because a shell variable is set ..

[Impala-ASF-CR] Don't assume that AUX exists just because a shell variable is set

2016-09-30 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: Don't assume that AUX exists just because a shell variable is set .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4563/2/tests/run-custom-c

[Impala-ASF-CR] Don't assume that AUX exists just because a shell variable is set

2016-09-30 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Don't assume that AUX exists just because a shell variable is set .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4563/1/tests/run-custom-cluster-t

[Impala-ASF-CR] Don't assume that AUX exists just because a shell variable is set

2016-09-30 Thread Jim Apple (Code Review)
Hello Michael Brown, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4563 to look at the new patch set (#2). Change subject: Don't assume that AUX exists just because a shell variable is set ..

[Impala-ASF-CR] Don't assume that AUX exists just because a shell variable is set

2016-09-29 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: Don't assume that AUX exists just because a shell variable is set .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4563 To unsubscri

[Impala-ASF-CR] Don't assume that AUX exists just because a shell variable is set

2016-09-29 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: Don't assume that AUX exists just because a shell variable is set .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4563 To unsub

[Impala-ASF-CR] Don't assume that AUX exists just because a shell variable is set

2016-09-29 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Don't assume that AUX exists just because a shell variable is set .. Patch Set 1: I tested this when AUX is present; it passed. I'm now testing when AUX is absent.

[Impala-ASF-CR] Don't assume that AUX exists just because a shell variable is set

2016-09-29 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Don't assume that AUX exists just because a shell variable is set .. Patch Set 1: Testing now. I expect the tests to take a few hours. -- To view, visit http://ge

[Impala-ASF-CR] Don't assume that AUX exists just because a shell variable is set

2016-09-29 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new change for review. http://gerrit.cloudera.org:8080/4563 Change subject: Don't assume that AUX exists just because a shell variable is set .. Don't assume that AUX exists just because a shell varia