[Impala-ASF-CR] Allow the SASL protocol service name to be configurable

2017-10-10 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8230 ) Change subject: Allow the SASL protocol service name to be configurable .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cl

[Impala-ASF-CR] [DOCS] Mention Kerberos and TLS for Kudu security

2017-04-14 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [DOCS] Mention Kerberos and TLS for Kudu security .. Patch Set 1: The link will be https://kudu.apache.org/docs/security.html, but it's not published quite yet.

[Impala-ASF-CR] IMPALA-4403: Implement SHOW RANGE PARTITIONS for Kudu tables

2016-12-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: IMPALA-4403: Implement SHOW RANGE PARTITIONS for Kudu tables .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5390/2/fe/src/main/java/org/apache/im

[Impala-ASF-CR] Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE

2016-12-06 Thread Dan Burkert (Code Review)
Hello Dimitris Tsirogiannis, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5382 to look at the new patch set (#3). Change subject: Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE ...

[Impala-ASF-CR] Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE

2016-12-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE .. Patch Set 2: PS2 build: http://sandbox.jenkins.cloudera.com/job/impala-umbrella-build-and-test/6174/co

[Impala-ASF-CR] Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE

2016-12-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/5382/1/docs/topics/impala_create_table.x

[Impala-ASF-CR] Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE

2016-12-06 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new patch set (#2). Change subject: Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE .. Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE This commit also removes the now unused `DISTRIB

[Impala-ASF-CR] Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE

2016-12-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE .. Patch Set 1: Kicked off a build here: http://sandbox.jenkins.cloudera.com/job/impala-umbrella-build-an

[Impala-ASF-CR] Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE

2016-12-06 Thread Dan Burkert (Code Review)
Hello Matthew Jacobs, Alex Behm, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5382 to review the following change. Change subject: Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE

[Impala-ASF-CR] IMPALA-3920: TotalStorageWaitTime counter not populated for fragments with Kudu scan node

2016-10-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: IMPALA-3920: TotalStorageWaitTime counter not populated for fragments with Kudu scan node .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4639/1/

[Impala-ASF-CR] IMPALA-3920: TotalStorageWaitTime counter not populated for fragments with Kudu scan node

2016-10-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: IMPALA-3920: TotalStorageWaitTime counter not populated for fragments with Kudu scan node .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/4639/1