[Impala-ASF-CR] IMPALA-5772: failure in test scratch disk.TestScratchDir didn't occur

2017-08-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new change for review. http://gerrit.cloudera.org:8080/7615 Change subject: IMPALA-5772: failure in test_scratch_disk.TestScratchDir didn't occur .. IMPALA-5772: failure in

[Impala-ASF-CR] IMPALA-5744: Add 'use krpc' flag and create DataStream interface

2017-08-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5744: Add 'use_krpc' flag and create DataStream interface .. Patch Set 10: Build started:

[Impala-ASF-CR] IMPALA-5661: buffer pool limit

2017-08-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5661: buffer pool limit .. Patch Set 5: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1014/ -- To view, visit

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

2017-08-07 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 5: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1013/ -- To view,

[Impala-ASF-CR] IMPALA-4674: Part 2.5: Rename BufferedTupleStreamV2

2017-08-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4674: Part 2.5: Rename BufferedTupleStreamV2 .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1012/ -- To

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

2017-08-07 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 6: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1011/

[Impala-ASF-CR] IMPALA-5768: Better developer documentation

2017-08-07 Thread Zach Amsden (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7350 to look at the new patch set (#5). Change subject: IMPALA-5768: Better developer documentation .. IMPALA-5768:

[Impala-ASF-CR] IMPALA-1478: Improve error message when subquery is used in the ON clause

2017-08-07 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has abandoned this change. Change subject: IMPALA-1478: Improve error message when subquery is used in the ON clause .. Abandoned Moved to CR:7588 -- To view, visit http://gerrit.cloudera.org:8080/7603

[Impala-ASF-CR] IMPALA-1478: Improve error message when subquery is used in the ON clause

2017-08-07 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-1478: Improve error message when subquery is used in the ON clause .. Patch Set 3: Code-Review+1 (2 comments) Adding Alex to do a +2 review.

[Impala-ASF-CR] IMPALA-5744: Add 'use krpc' flag and create DataStream interface

2017-08-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5744: Add 'use_krpc' flag and create DataStream interface .. Patch Set 10: Build started:

[Impala-ASF-CR] IMPALA-5774: Prevent FindInSet() from reading off end of string

2017-08-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5774: Prevent FindInSet() from reading off end of string .. IMPALA-5774: Prevent FindInSet() from reading off end of string Change-Id:

[Impala-ASF-CR] IMPALA-5744: Add 'use krpc' flag and create DataStream interface

2017-08-07 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-5744: Add 'use_krpc' flag and create DataStream interface .. Patch Set 10: Code-Review+2 Thanks for the review! Carry +2 -- To view, visit

[Impala-ASF-CR] IMPALA-5774: Prevent FindInSet() from reading off end of string

2017-08-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5774: Prevent FindInSet() from reading off end of string .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7608 To

[Impala-ASF-CR] IMPALA-1478: Improve error message when subquery is used in the ON clause

2017-08-07 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#3). Change subject: IMPALA-1478: Improve error message when subquery is used in the ON clause .. IMPALA-1478: Improve error message when subquery is used in the ON clause Fix: Print

[Impala-ASF-CR] IMPALA-5744: Add 'use krpc' flag and create DataStream interface

2017-08-07 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-5744: Add 'use_krpc' flag and create DataStream interface .. Patch Set 9: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7542 To

[Impala-ASF-CR] IMPALA-5661: buffer pool limit

2017-08-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5661: buffer pool limit .. Patch Set 5: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1009/ -- To view, visit

[Impala-ASF-CR] IMPALA-5661: buffer pool limit

2017-08-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5661: buffer pool limit .. Patch Set 5: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1008/ -- To view, visit

[Impala-ASF-CR] IMPALA-5661: buffer pool limit

2017-08-07 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5661: buffer pool limit .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7462 To unsubscribe, visit

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

2017-08-07 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 5: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1007/ -- To view,

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

2017-08-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5572: Timestamp codegen for text scanner .. Patch Set 5: It looks like you're hitting a known-flaky test:

[Impala-ASF-CR] IMPALA-5661: buffer pool limit

2017-08-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5661: buffer pool limit .. Patch Set 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/7462/4//COMMIT_MSG Commit Message: Line 7: IMPALA-5661: buffer

[Impala-ASF-CR] IMPALA-5661: buffer pool limit

2017-08-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#5). Change subject: IMPALA-5661: buffer pool limit .. IMPALA-5661: buffer pool limit Adds the --buffer_pool_limit flag to control the buffer pool size. It can be specified as either

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

2017-08-07 Thread Tianyi Wang (Code Review)
Tianyi Wang has posted comments on this change. Change subject: IMPALA-5572: Timestamp codegen for text scanner .. Patch Set 5: I will test the rebased version again. -- To view, visit http://gerrit.cloudera.org:8080/7556 To

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

2017-08-07 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 5: Verified-1 Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/1002/ --

[Impala-ASF-CR] IMPALA-5715: (mitigation only) defer destruction of MemTrackers

2017-08-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5715: (mitigation only) defer destruction of MemTrackers .. Patch Set 4: (5 comments) Addressed the comments. Was in the middle of a rebase onto the

[Impala-ASF-CR] IMPALA-5715: (mitigation only) defer destruction of MemTrackers

2017-08-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5715: (mitigation only) defer destruction of MemTrackers .. Patch Set 4: Going to run tests under ASAN before merging -- To view, visit

[Impala-ASF-CR] IMPALA-5715: (mitigation only) defer destruction of MemTrackers

2017-08-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5715: (mitigation only) defer destruction of MemTrackers .. Patch Set 4: Code-Review+2 Carry +2 -- To view, visit http://gerrit.cloudera.org:8080/7492

[Impala-ASF-CR] IMPALA-4674: Part 2.5: Rename BufferedTupleStreamV2

2017-08-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4674: Part 2.5: Rename BufferedTupleStreamV2 .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1006/ -- To

[Impala-ASF-CR] IMPALA-4674: Part 2.5: Rename BufferedTupleStreamV2

2017-08-07 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4674: Part 2.5: Rename BufferedTupleStreamV2 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7609 To unsubscribe, visit

[Impala-ASF-CR] (PREVIEW) IMPALA-5684: Optionally run be tests in sharded mode

2017-08-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: (PREVIEW) IMPALA-5684: Optionally run be tests in sharded mode .. Patch Set 2: (5 comments) http://gerrit.cloudera.org:8080/#/c/7467/2//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-5661: buffer pool limit

2017-08-07 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5661: buffer pool limit .. Patch Set 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/7462/4//COMMIT_MSG Commit Message: Line 7: IMPALA-5661:

[Impala-ASF-CR] IMPALA-5715: (mitigation only) defer destruction of MemTrackers

2017-08-07 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-5715: (mitigation only) defer destruction of MemTrackers .. Patch Set 4: (2 comments)

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

2017-08-07 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 6: Code-Review+2 Forgot to add the couple of lines of code to enable this in the

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

2017-08-07 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 6: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1005/

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

2017-08-07 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 (#6). Change subject: IMPALA-5696: Enable cipher configuration when using TLS / Thrift

[Impala-ASF-CR] IMPALA-5715: (mitigation only) defer destruction of MemTrackers

2017-08-07 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-5715: (mitigation only) defer destruction of MemTrackers .. Patch Set 4: Code-Review+2 (1 comment)

[Impala-ASF-CR] (PREVIEW) IMPALA-5684: Optionally run be tests in sharded mode

2017-08-07 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: (PREVIEW) IMPALA-5684: Optionally run be tests in sharded mode .. Patch Set 2: Only in the sense that I was hoping for some feedback on the approach (e.g. is

[Impala-ASF-CR] (PREVIEW) IMPALA-5684: Optionally run be tests in sharded mode

2017-08-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: (PREVIEW) IMPALA-5684: Optionally run be tests in sharded mode .. Patch Set 2: Is this still a preview only? -- To view, visit

[Impala-ASF-CR] IMPALA-5715: (mitigation only) defer destruction of MemTrackers

2017-08-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5715: (mitigation only) defer destruction of MemTrackers .. Patch Set 4: (3 comments)

[Impala-ASF-CR] IMPALA-5715: (mitigation only) defer destruction of MemTrackers

2017-08-07 Thread Tim Armstrong (Code Review)
Hello Matthew Jacobs, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7492 to look at the new patch set (#4). Change subject: IMPALA-5715: (mitigation only) defer destruction of MemTrackers

[Impala-ASF-CR] IMPALA-5661: buffer pool limit

2017-08-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#4). Change subject: IMPALA-5661: buffer pool limit .. IMPALA-5661: buffer pool limit Testing: Started up a cluster with --buffer_pool_limit=10%, confirmed via /metrics page that the

[Impala-ASF-CR] IMPALA-4674: Part 2.5: Rename BufferedTupleStreamV2

2017-08-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new change for review. http://gerrit.cloudera.org:8080/7609 Change subject: IMPALA-4674: Part 2.5: Rename BufferedTupleStreamV2 .. IMPALA-4674: Part 2.5: Rename BufferedTupleStreamV2 This is

[Impala-ASF-CR] IMPALA-5768: Better developer documentation

2017-08-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5768: Better developer documentation .. Patch Set 4: Code-Review+1 (1 comment) LGTM. I'll give other reviewers a chance to weigh in.

[Impala-ASF-CR] IMPALA-5743: Support TLS version configuration for Thrift servers

2017-08-07 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-5743: Support TLS version configuration for Thrift servers .. Patch Set 1: (6 comments)

[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-5546: Allow creating unpartitioned Kudu tables

2017-08-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5546: Allow creating unpartitioned Kudu tables .. IMPALA-5546: Allow creating unpartitioned Kudu tables This patch makes it possible to

[Impala-ASF-CR] IMPALA-5546: Allow creating unpartitioned Kudu tables

2017-08-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5546: Allow creating unpartitioned Kudu tables .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7446 To unsubscribe,

[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-5696: Enable cipher configuration when using TLS / Thrift

2017-08-07 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/1004/

[Impala-ASF-CR] IMPALA-5774: Prevent FindInSet() from reading off end of string

2017-08-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5774: Prevent FindInSet() from reading off end of string .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1003/

[Impala-ASF-CR] IMPALA-5774: Prevent FindInSet() from reading off end of string

2017-08-07 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-5774: Prevent FindInSet() from reading off end of string .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7608 To

[Impala-ASF-CR] IMPALA-5774: Prevent FindInSet() from reading off end of string

2017-08-07 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new change for review. http://gerrit.cloudera.org:8080/7608 Change subject: IMPALA-5774: Prevent FindInSet() from reading off end of string .. IMPALA-5774: Prevent FindInSet() from reading off

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

2017-08-07 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 failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/1000/

[Impala-ASF-CR] IMPALA-5715: (mitigation only) defer destruction of MemTrackers

2017-08-07 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-5715: (mitigation only) defer destruction of MemTrackers .. Patch Set 3: (3 comments) LGTM. Please address the question in ~MemTracker().

[Impala-ASF-CR] IMPALA-5743: Support TLS version configuration for Thrift servers

2017-08-07 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new change for review. http://gerrit.cloudera.org:8080/7606 Change subject: IMPALA-5743: Support TLS version configuration for Thrift servers .. IMPALA-5743: Support TLS version configuration

[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-1478: Improve error message when subquery is used in the ON clause

2017-08-07 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-1478: Improve error message when subquery is used in the ON clause .. Patch Set 1: Please squash this commit into

[native-toolchain-CR] Bump Kudu version to 943b1ae

2017-08-07 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has submitted this change and it was merged. Change subject: Bump Kudu version to 943b1ae .. Bump Kudu version to 943b1ae Change-Id: I15c1f0a9ff0002b7bda6f43e2fd5156ef21a04e6 --- M buildall.sh 1 file changed, 1

[native-toolchain-CR] Bump Kudu version to 943b1ae

2017-08-07 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Bump Kudu version to 943b1ae .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7601 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-1478: Improve error message when subquery is used in the ON clause

2017-08-07 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#2). Change subject: IMPALA-1478: Improve error message when subquery is used in the ON clause .. IMPALA-1478: Improve error message when subquery is used in the ON clause Fix: Print

[Impala-ASF-CR] IMPALA-1478: Improve error message when subquery is used in the ON clause

2017-08-07 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new change for review. http://gerrit.cloudera.org:8080/7603 Change subject: IMPALA-1478: Improve error message when subquery is used in the ON clause .. IMPALA-1478: Improve error message when

[native-toolchain-CR] Bump Kudu version to 943b1ae

2017-08-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to 943b1ae .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7601 To unsubscribe, visit

[native-toolchain-CR] Bump Kudu version to 943b1ae

2017-08-07 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/7601 Change subject: Bump Kudu version to 943b1ae .. Bump Kudu version to 943b1ae Change-Id: I15c1f0a9ff0002b7bda6f43e2fd5156ef21a04e6 ---

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

2017-08-07 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 5: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1002/ -- To view,

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

2017-08-07 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5572: Timestamp codegen for text scanner .. Patch Set 5: Code-Review+2 rebase. -- To view, visit http://gerrit.cloudera.org:8080/7556 To unsubscribe,

[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

[Impala-ASF-CR] IMPALA-5744: Add 'use krpc' flag and create DataStream interface

2017-08-07 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-5744: Add 'use_krpc' flag and create DataStream interface .. Patch Set 8: Code-Review+2 (8 comments) Thanks for the review. Will also wait for Henry's

[Impala-ASF-CR] IMPALA-5744: Add 'use krpc' flag and create DataStream interface

2017-08-07 Thread Sailesh Mukil (Code Review)
Hello Michael Ho, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7542 to look at the new patch set (#9). Change subject: IMPALA-5744: Add 'use_krpc' flag and create DataStream interface

[Impala-ASF-CR] IMPALA-5546: Allow creating unpartitioned Kudu tables

2017-08-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5546: Allow creating unpartitioned Kudu tables .. Patch Set 4: Code-Review+2 (1 comment) Carrying forward.

[Impala-ASF-CR] IMPALA-5546: Allow creating unpartitioned Kudu tables

2017-08-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5546: Allow creating unpartitioned Kudu tables .. Patch Set 4: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1001/ -- To

[Impala-ASF-CR] IMPALA-5546: Allow creating unpartitioned Kudu tables

2017-08-07 Thread Thomas Tauber-Marshall (Code Review)
Hello Impala Public Jenkins, Matthew Jacobs, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7446 to look at the new patch set (#4). Change subject: IMPALA-5546: Allow creating unpartitioned Kudu tables

[Impala-ASF-CR] IMPALA-5749: coordinator race hits DCHECK 'num remaining backends > 0'

2017-08-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5749: coordinator race hits DCHECK 'num_remaining_backends_ > 0' .. Patch Set 2: Code-Review+1 Carrying forward +1 -- To view, visit

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

2017-08-07 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/1000/

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

2017-08-07 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: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7600 To

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

2017-08-07 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. 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 vote Change-Id:

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

2017-08-07 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: Verified-1 Build failed: