[kudu-CR] [tools] command to reset authz cache in kudu CLI

2019-05-03 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13139 ) Change subject: [tools] command to reset authz cache in kudu CLI .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/13139/

[kudu-CR] [tools] command to reset authz cache in kudu CLI

2019-05-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13139 ) Change subject: [tools] command to reset authz cache in kudu CLI .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/1313

[kudu-CR] [docs] KUDU-2808: Correct doc for default compression

2019-05-03 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13217 ) Change subject: [docs] KUDU-2808: Correct doc for default compression .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/1

[kudu-CR] [tools] command to reset authz cache in kudu CLI

2019-05-03 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13139 ) Change subject: [tools] command to reset authz cache in kudu CLI .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/13139/

[kudu-CR] [tools] command to reset authz cache in kudu CLI

2019-05-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13139 ) Change subject: [tools] command to reset authz cache in kudu CLI .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/1313

[kudu-CR] [tools] command to reset authz cache in kudu CLI

2019-05-03 Thread Alexey Serbin (Code Review)
Hello Andrew Wong, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13139 to look at the new patch set (#5). Change subject: [tools] command to reset authz cache in kudu CLI

[kudu-CR] [tools] command to reset authz cache in kudu CLI

2019-05-03 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13139 ) Change subject: [tools] command to reset authz cache in kudu CLI .. Patch Set 4: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/

[kudu-CR] [tools] command to reset authz cache in kudu CLI

2019-05-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13139 ) Change subject: [tools] command to reset authz cache in kudu CLI .. Patch Set 4: Verified+1 unrelated flakes in: * HmsSentryConfigur

[kudu-CR] [tools] command to reset authz cache in kudu CLI

2019-05-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/13139 ) Change subject: [tools] command to reset authz cache in kudu CLI .. Removed reviewer Kudu Jenkins with the following votes: *

[kudu-CR] [tools] command to reset authz cache in kudu CLI

2019-05-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13139 ) Change subject: [tools] command to reset authz cache in kudu CLI .. Patch Set 4: (3 comments) http://gerrit.cloudera.org:8080/#/c/131

[kudu-CR] [tools] command to reset authz cache in kudu CLI

2019-05-03 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Andrew Wong, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13139 to look at the new patch set (#4). Change subject: [tools] command to reset authz cache in kudu CLI ..

[kudu-CR] [backup] Support table name alterations between Kudu backups

2019-05-03 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13210 ) Change subject: [backup] Support table name alterations between Kudu backups .. Patch Set 4: Code-Review+2 -- To view, visit http://gerr

[kudu-CR] [authz] validator for --sentry service rpc addresses flag

2019-05-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13227 ) Change subject: [authz] validator for --sentry_service_rpc_addresses flag .. [authz] validator for --sentry_service_rpc_address

[kudu-CR] [authz] validator for --sentry service rpc addresses flag

2019-05-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/13227 ) Change subject: [authz] validator for --sentry_service_rpc_addresses flag .. Removed reviewer Kudu Jenkins with the following v

[kudu-CR] [authz] validator for --sentry service rpc addresses flag

2019-05-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13227 ) Change subject: [authz] validator for --sentry_service_rpc_addresses flag .. Patch Set 3: Verified+1 (2 comments) unrelated flakes in

[kudu-CR] [authz] validator for --sentry service rpc addresses flag

2019-05-03 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13227 ) Change subject: [authz] validator for --sentry_service_rpc_addresses flag .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.

[kudu-CR] [authz] validator for --sentry service rpc addresses flag

2019-05-03 Thread Alexey Serbin (Code Review)
Hello Andrew Wong, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13227 to look at the new patch set (#3). Change subject: [authz] validator for --sentry_service_rpc_addresses flag ...

[kudu-CR] [docs] KUDU-2808: Correct doc for default compression

2019-05-03 Thread Mitch Barnett (Code Review)
Mitch Barnett has posted comments on this change. ( http://gerrit.cloudera.org:8080/13217 ) Change subject: [docs] KUDU-2808: Correct doc for default compression .. Patch Set 1: (1 comment) > Patch Set 1: > > (1 comment) htt

[kudu-CR] [docs] KUDU-2808: Correct doc for default compression

2019-05-03 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13217 ) Change subject: [docs] KUDU-2808: Correct doc for default compression .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/1

[kudu-CR] [authz] validator for --sentry service rpc addresses flag

2019-05-03 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13227 ) Change subject: [authz] validator for --sentry_service_rpc_addresses flag .. Patch Set 2: Code-Review+1 Looks good modulo Adar's nits.

[kudu-CR] authz: mark tserver flag non-experimental

2019-05-03 Thread Andrew Wong (Code Review)
Andrew Wong has removed a vote on this change. Change subject: authz: mark tserver flag non-experimental .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13225 To unsubscribe, visit h

[kudu-CR] authz: mark tserver flag non-experimental

2019-05-03 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13225 ) Change subject: authz: mark tserver flag non-experimental .. Patch Set 1: Verified+1 Failure was TestKuduBackup and DefaultSourceTest.

[kudu-CR] authz: mark tserver flag non-experimental

2019-05-03 Thread Andrew Wong (Code Review)
Andrew Wong has removed a vote on this change. Change subject: authz: mark tserver flag non-experimental .. Removed -Verified by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13225 To unsubscribe, visit ht

[kudu-CR] authz: mark tserver flag non-experimental

2019-05-03 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13225 ) Change subject: authz: mark tserver flag non-experimental .. authz: mark tserver flag non-experimental This is a follow-up to b0

[kudu-CR] authz: mark tserver flag non-experimental

2019-05-03 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13225 ) Change subject: authz: mark tserver flag non-experimental .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:808

[kudu-CR] [authz] validator for --sentry service rpc addresses flag

2019-05-03 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13227 ) Change subject: [authz] validator for --sentry_service_rpc_addresses flag .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#

[kudu-CR] HIVE-12971 Add a Hive serde for Kudu

2019-05-03 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13228 ) Change subject: HIVE-12971 Add a Hive serde for Kudu .. Patch Set 8: > Patch Set 8: > > The issue being that there's too much static sta

[kudu-CR] [backup] Support table name alterations between Kudu backups

2019-05-03 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13210 ) Change subject: [backup] Support table name alterations between Kudu backups .. Patch Set 4: Code-Review+1 -- To view, visit http://gerr

[kudu-CR] HIVE-12971 Add a Hive serde for Kudu

2019-05-03 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13228 ) Change subject: HIVE-12971 Add a Hive serde for Kudu .. Patch Set 8: > Yes, I had a look at it and it turns out kudu-mapreduce doesn't su

[kudu-CR] [HIVE-12971] add a hive serde for kudu

2019-05-03 Thread Clemens Valiente (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13230 to look at the new patch set (#2). Change subject: [HIVE-12971] add a hive serde for kudu .. [HIVE-12971] add a hiv

[kudu-CR] HIVE-12971 Add a Hive serde for Kudu

2019-05-03 Thread Clemens Valiente (Code Review)
Clemens Valiente has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13231 Change subject: HIVE-12971 Add a Hive serde for Kudu .. HIVE-12971 Add a Hive serde for Kudu Change-Id: Ibdc5bdf671a29b566cfb8bc790a

[kudu-CR] HIVE-12971 Add a Hive serde for Kudu

2019-05-03 Thread Clemens Valiente (Code Review)
Clemens Valiente has abandoned this change. ( http://gerrit.cloudera.org:8080/13231 ) Change subject: HIVE-12971 Add a Hive serde for Kudu .. Abandoned Duplicate, see https://gerrit.cloudera.org/#/c/13230/ -- To view, visit ht

[kudu-CR] HIVE-12971 Add a Hive serde for Kudu

2019-05-03 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13228 ) Change subject: HIVE-12971 Add a Hive serde for Kudu .. Patch Set 8: (8 comments) http://gerrit.cloudera.org:8080/#/c/13228/7//COMMIT_MS

[kudu-CR] HIVE-12971 Add a Hive serde for Kudu

2019-05-03 Thread Clemens Valiente (Code Review)
Clemens Valiente has abandoned this change. ( http://gerrit.cloudera.org:8080/13228 ) Change subject: HIVE-12971 Add a Hive serde for Kudu .. Abandoned not fixing kudu-mapreduce further -- To view, visit http://gerrit.cloudera

[kudu-CR] HIVE-12971 Add a Hive serde for Kudu

2019-05-03 Thread Clemens Valiente (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13231 to look at the new patch set (#2). Change subject: HIVE-12971 Add a Hive serde for Kudu .. HIVE-12971 Add a Hive se

[kudu-CR] HIVE-12971 Add a Hive serde for Kudu

2019-05-03 Thread Clemens Valiente (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13228 to look at the new patch set (#8). Change subject: HIVE-12971 Add a Hive serde for Kudu .. HIVE-12971 A

[kudu-CR] [HIVE-12971] add a hive serde for kudu

2019-05-03 Thread Clemens Valiente (Code Review)
Clemens Valiente has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13230 Change subject: [HIVE-12971] add a hive serde for kudu .. [HIVE-12971] add a hive serde for kudu Change-Id: I72daf3c3f06ccf67e3fa303

[kudu-CR] Throw exception on write error with kudu-mapreduce

2019-05-03 Thread Clemens Valiente (Code Review)
Clemens Valiente has posted comments on this change. ( http://gerrit.cloudera.org:8080/13228 ) Change subject: Throw exception on write error with kudu-mapreduce .. Patch Set 7: Yes, I had a look at it and it turns out kudu-ma

[kudu-CR] Throw exception on write error with kudu-mapreduce

2019-05-03 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13228 ) Change subject: Throw exception on write error with kudu-mapreduce .. Patch Set 7: Looks like there are two Java test failures to fix her

[kudu-CR] [authz] validator for --sentry service rpc addresses flag

2019-05-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/13227 ) Change subject: [authz] validator for --sentry_service_rpc_addresses flag .. Removed reviewer Kudu Jenkins with the following v

[kudu-CR] [authz] validator for --sentry service rpc addresses flag

2019-05-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13227 ) Change subject: [authz] validator for --sentry_service_rpc_addresses flag .. Patch Set 2: Verified+1 unrelated flake in Java tests:

[kudu-CR] [backup] Support table name alterations between Kudu backups

2019-05-03 Thread Grant Henke (Code Review)
Hello Mike Percy, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13210 to look at the new patch set (#4). Change subject: [backup] Support table name alterations between Kudu backups

[kudu-CR] Throw exception on write error with kudu-mapreduce

2019-05-03 Thread Clemens Valiente (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13228 to look at the new patch set (#6). Change subject: Throw exception on write error with kudu-mapreduce .. Throw exce

[kudu-CR] Throw exception on write error with kudu-mapreduce

2019-05-03 Thread Clemens Valiente (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13228 to look at the new patch set (#5). Change subject: Throw exception on write error with kudu-mapreduce .. Throw exce

[kudu-CR] Throw exception on write errors with the kudu-mapreduce client

2019-05-03 Thread Clemens Valiente (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13228 to look at the new patch set (#2). Change subject: Throw exception on write errors with the kudu-mapreduce client ..

[kudu-CR] Throw exception on write errors with kudu-mapreduce

2019-05-03 Thread Clemens Valiente (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13228 to look at the new patch set (#3). Change subject: Throw exception on write errors with kudu-mapreduce .. Throw exc

[kudu-CR] Throw exception on write error with kudu-mapreduce

2019-05-03 Thread Clemens Valiente (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13228 to look at the new patch set (#4). Change subject: Throw exception on write error with kudu-mapreduce .. Throw exce

[kudu-CR] throw error on write errors

2019-05-03 Thread Clemens Valiente (Code Review)
Clemens Valiente has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13228 Change subject: throw error on write errors .. throw error on write errors Change-Id: I6db6a59104425c396fbbeb2105bf2c777cc348c7 ---

[kudu-CR] authz: mark tserver flag non-experimental

2019-05-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13225 ) Change subject: authz: mark tserver flag non-experimental .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:

[kudu-CR] [authz] validator for --sentry service rpc addresses flag

2019-05-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13227 Change subject: [authz] validator for --sentry_service_rpc_addresses flag .. [authz] validator for --sentry_service_rpc_addresses flag

[kudu-CR] [authz] validator for --sentry service rpc addresses flag

2019-05-03 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Andrew Wong, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13227 to look at the new patch set (#2). Change subject: [authz] validator for --sentry_service_rpc_addresses flag .