[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-10 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 6: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7587 To unsubscribe,

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-10 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. IMPALA-4407: Move Impala setup procedures to main repo Before this change, Impala has

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-10 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 6: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1030/ -- To vi

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-10 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 6: > Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/1029/ https://issues.apach

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-10 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 6: Verified-1 Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/1029/

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-10 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 6: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1029/ -- To vi

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-10 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 6: Code-Review+2 rebase carry +2 -- To view, visit http://gerrit.cloudera.org:8080/7587 To unsu

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-10 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 5: Code-Review+2 Thanks for doing this! LGTM. -- To view, visit http://gerrit.cloudera.org:

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-10 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 5: Sailesh since you also took a look at this and left comments, do you want to give the +2

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-10 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 5: Code-Review+1 (1 comment) Carry +1s from Michael B. and Lars. http://gerrit.cloudera.org:808

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-10 Thread Jim Apple (Code Review)
Hello Michael Brown, Lars Volker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7587 to look at the new patch set (#5). Change subject: IMPALA-4407: Move Impala setup procedures to main repo

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-10 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 4: Code-Review+1 (2 comments) http://gerrit.cloudera.org:8080/#/c/7587/3/bin/bootstrap_devel

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-10 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 3: (11 comments) http://gerrit.cloudera.org:8080/#/c/7587/3/bin/bootstrap_development.sh File bi

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-10 Thread Jim Apple (Code Review)
Hello Lars Volker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7587 to look at the new patch set (#4). Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. IMPALA-

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-07 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/7587/3/bin/bootstrap_development.sh File

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-07 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 3: > I'm testing this out. It worked pretty well on the Ubuntu 16 system I tried out. I was

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-07 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/7587/3/bin/bootstrap_development.sh File

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-07 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/7587/3/bin/bootstrap_development.sh File

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-07 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 3: (7 comments) http://gerrit.cloudera.org:8080/#/c/7587/3/bin/bootstrap_development.sh File

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-07 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/7587/3/bin/bootstrap_development.sh File

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-06 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7587 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-06 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7587/2/bin/bootstrap_development.sh File bin/

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-06 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/7587/2/bin/bootstrap_development.sh File b

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-06 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 2: (10 comments) http://gerrit.cloudera.org:8080/#/c/7587/2/bin/bootstrap_development.sh File bi

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-06 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new patch set (#3). Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. IMPALA-4407: Move Impala setup procedures to main repo Before this change, Impala has relied on a chef setup

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-05 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 2: (10 comments) Thank you for creating this! http://gerrit.cloudera.org:8080/#/c/7587/2/bin/

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-05 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new patch set (#2). Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. IMPALA-4407: Move Impala setup procedures to main repo Before this change, Impala has relied on a chef setup

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-04 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 1: > I'm testing this out. https://jenkins.impala.io/job/ubuntu-16.04-from-scratch/13 -- To vi

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-04 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7587/1/bin/bootstrap_development.sh File bin/

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-04 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 1: I'm testing this out. -- To view, visit http://gerrit.cloudera.org:8080/7587 To unsubscr

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-04 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7587/1/bin/bootstrap_development.sh File

[Impala-ASF-CR] IMPALA-4407: Move Impala setup procedures to main repo

2017-08-04 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new change for review. http://gerrit.cloudera.org:8080/7587 Change subject: IMPALA-4407: Move Impala setup procedures to main repo .. IMPALA-4407: Move Impala setup procedures to main repo Before this