[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-09-05 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 2: Need to get back to this now that I have smokes fixed. I imagine I'll want to do things a

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-09-05 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 2: What's the next step on this one? -- To view, visit http://gerrit.cloudera.org:8080/7581 To

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-15 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7581/2/bin/impala-config.sh File

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-15 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7581/2/bin/impala-config.sh File

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-14 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7581/2/bin/impala-config.sh File

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-14 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 2: (4 comments) A couple of minor nits. Also, let me make sure I didn't accidentally introduce

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-14 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7581 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-10 Thread Zach Amsden (Code Review)
Zach Amsden has abandoned this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Abandoned Gerrit created another change for some reason. -- To view, visit http://gerrit.cloudera.org:8080/7648 To

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-10 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#2). Change subject: IMPALA-5764: Allow overriding packaged components .. IMPALA-5764: Allow overriding packaged components For allowing multiple different distributions to build

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-10 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new change for review. http://gerrit.cloudera.org:8080/7648 Change subject: IMPALA-5764: Allow overriding packaged components .. IMPALA-5764: Allow overriding packaged components For allowing

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-10 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 1: needs rebase against merged changes, coming presently -- To view, visit

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-09 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 1: Ping? -- To view, visit http://gerrit.cloudera.org:8080/7581 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-05 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7581/1//COMMIT_MSG Commit Message: Line 9: For

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-04 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/7581/1//COMMIT_MSG Commit Message: Line 9:

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-04 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7581/1/bin/impala-config.sh File

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-04 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/7581/1//COMMIT_MSG Commit Message: Line 9:

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-04 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7581/1//COMMIT_MSG Commit Message: Line 14:

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-04 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5764: Allow overriding packaged components .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/7581/1//COMMIT_MSG Commit Message: Line

[Impala-ASF-CR] IMPALA-5764: Allow overriding packaged components

2017-08-03 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new change for review. http://gerrit.cloudera.org:8080/7581 Change subject: IMPALA-5764: Allow overriding packaged components .. IMPALA-5764: Allow overriding packaged components For allowing