[Impala-ASF-CR] IMPALA-5484: Fix LICENSE issues discovered by IPMC in 2.9 vote

2017-08-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5484: Fix LICENSE issues discovered by IPMC in 2.9 vote .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/999/

[Impala-ASF-CR] IMPALA-5696: Enable cipher configuration when using TLS / Thrift

2017-08-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5696: Enable cipher configuration when using TLS / Thrift .. Patch Set 5: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/998/

[Impala-ASF-CR] IMPALA-5696: Enable cipher configuration when using TLS / Thrift

2017-08-06 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-5696: Enable cipher configuration when using TLS / Thrift .. Patch Set 5: Code-Review+2 Fix some clang-tidy warnings in the tests. -- To view, visit

[Impala-ASF-CR] IMPALA-5484: Fix LICENSE issues discovered by IPMC in 2.9 vote

2017-08-06 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-5484: Fix LICENSE issues discovered by IPMC in 2.9 vote .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7600 To

[Impala-ASF-CR] IMPALA-5696: Enable cipher configuration when using TLS / Thrift

2017-08-06 Thread Henry Robinson (Code Review)
Hello Impala Public Jenkins, Matthew Jacobs, Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7524 to look at the new patch set (#5). Change subject: IMPALA-5696: Enable cipher configuration when using TLS / Thrift

[Impala-ASF-CR] IMPALA-5484: Fix LICENSE issues discovered by IPMC in 2.9 vote

2017-08-06 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new change for review. http://gerrit.cloudera.org:8080/7600 Change subject: IMPALA-5484: Fix LICENSE issues discovered by IPMC in 2.9 vote .. IMPALA-5484: Fix LICENSE issues discovered by IPMC in 2.9

[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

[Impala-ASF-CR] IMPALA-5572: Timestamp codegen for text scanner

2017-08-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5572: Timestamp codegen for text scanner .. Patch Set 4: Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/997/ -- To view,

[Impala-ASF-CR] IMPALA-3200: more buffer pool end-to-end tests

2017-08-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-3200: more buffer pool end-to-end tests .. Patch Set 7: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7552 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3200: more buffer pool end-to-end tests

2017-08-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-3200: more buffer pool end-to-end tests .. IMPALA-3200: more buffer pool end-to-end tests This adds most of the end-to-end tests

[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

[Impala-ASF-CR] IMPALA-5696: Enable cipher configuration when using TLS / Thrift

2017-08-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5696: Enable cipher configuration when using TLS / Thrift .. Patch Set 4: Verified-1 Build failed:

[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

[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-5598: Fix excessive dumping in MemLimitExceeded

2017-08-06 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5598: Fix excessive dumping in MemLimitExceeded .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7597/1/be/src/runtime/mem-tracker.cc

[Impala-ASF-CR] IMPALA-5598: Fix excessive dumping in MemLimitExceeded

2017-08-06 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5598: Fix excessive dumping in MemLimitExceeded .. Patch Set 1: (3 comments) Looks good, just a couple of minor things.

[Impala-ASF-CR] IMPALA-5572: Timestamp codegen for text scanner

2017-08-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5572: Timestamp codegen for text scanner .. Patch Set 4: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/997/ -- To view,

[Impala-ASF-CR] IMPALA-3200: more buffer pool end-to-end tests

2017-08-06 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3200: more buffer pool end-to-end tests .. Patch Set 7: Code-Review+2 Rebase -- To view, visit http://gerrit.cloudera.org:8080/7552 To unsubscribe,

[Impala-ASF-CR] IMPALA-3200: more buffer pool end-to-end tests

2017-08-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-3200: more buffer pool end-to-end tests .. Patch Set 7: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/996/ -- To view,

[Impala-ASF-CR] IMPALA-5696: Enable cipher configuration when using TLS / Thrift

2017-08-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5696: Enable cipher configuration when using TLS / Thrift .. Patch Set 4: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/995/

[Impala-ASF-CR] IMPALA-5696: Enable cipher configuration when using TLS / Thrift

2017-08-06 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5696: Enable cipher configuration when using TLS / Thrift .. Patch Set 3: Code-Review+2 (1 comment) Makes sense. I like the refactoring; we should