[kudu-CR] [docs] Fix formatting in administration

2019-04-26 Thread Attila Bukor (Code Review)
Hello Will Berkeley, Alex Rodoni, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/13129 to review the following change. Change subject: [docs] Fix formatting in administration .. [docs] Fix

[kudu-CR] Make HiveMetastoreConfig class public

2019-04-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13128 ) Change subject: Make HiveMetastoreConfig class public .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080

[kudu-CR] hms: remove experimental tag from hms flags

2019-04-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13124 ) Change subject: hms: remove experimental tag from hms flags .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.or

[kudu-CR] [backup] Add more metadata fields

2019-04-26 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13130 Change subject: [backup] Add more metadata fields .. [backup] Add more metadata fields This patch adds more metadata fields to the metdat

[kudu-CR] hms: remove experimental tag from hms flags

2019-04-26 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13124 ) Change subject: hms: remove experimental tag from hms flags .. Patch Set 2: Verified+1 Test Result (2 failures / +1) MultiThreadedHybr

[kudu-CR] hms: remove experimental tag from hms flags

2019-04-26 Thread Andrew Wong (Code Review)
Andrew Wong has removed a vote on this change. Change subject: hms: remove experimental tag from hms flags .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13124 To unsubscribe, visit

[kudu-CR] Make HiveMetastoreConfig class public

2019-04-26 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13128 ) Change subject: Make HiveMetastoreConfig class public .. Patch Set 1: Verified+1 Test Result (1 failure / +1) RemoteKsckTest.TestClust

[kudu-CR] Make HiveMetastoreConfig class public

2019-04-26 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13128 ) Change subject: Make HiveMetastoreConfig class public .. Make HiveMetastoreConfig class public Commit 2c572cbd4 introduced the J

[kudu-CR] Make HiveMetastoreConfig class public

2019-04-26 Thread Andrew Wong (Code Review)
Andrew Wong has removed a vote on this change. Change subject: Make HiveMetastoreConfig class public .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13128 To unsubscribe, visit http:

[kudu-CR] [backup] Ensure the last metadata is restored

2019-04-26 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13132 Change subject: [backup] Ensure the last metadata is restored .. [backup] Ensure the last metadata is restored When creating a table, ens

[kudu-CR] Make HiveMetastoreConfig class public

2019-04-26 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13128 ) Change subject: Make HiveMetastoreConfig class public .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/1

[kudu-CR] hms: remove experimental tag from hms flags

2019-04-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13124 ) Change subject: hms: remove experimental tag from hms flags .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.or

[kudu-CR] [docs] Fix formatting in administration

2019-04-26 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/13129 ) Change subject: [docs] Fix formatting in administration .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080

[kudu-CR] [docs] Fix formatting in administration

2019-04-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13129 ) Change subject: [docs] Fix formatting in administration .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:80

[kudu-CR] Add PySpark info to Developing Doc

2019-04-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13102 ) Change subject: Add PySpark info to Developing Doc .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/13102/2/docs/devel

[kudu-CR] Add PySpark info to Developing Doc

2019-04-26 Thread Anonymous Coward (Code Review)
a...@phdata.io has posted comments on this change. ( http://gerrit.cloudera.org:8080/13102 ) Change subject: Add PySpark info to Developing Doc .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/13102/2/docs/deve

[kudu-CR] hms: remove experimental tag from hms flags

2019-04-26 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13124 ) Change subject: hms: remove experimental tag from hms flags .. Patch Set 2: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/

[kudu-CR] [backup] Add more metadata fields

2019-04-26 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13130 ) Change subject: [backup] Add more metadata fields .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/13130/1//COMMIT_MSG C

[kudu-CR] hms: remove experimental tag from hms flags

2019-04-26 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13124 ) Change subject: hms: remove experimental tag from hms flags .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/13124/2/src

[kudu-CR] sentry: remove experimental tag from sentry flags

2019-04-26 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13133 Change subject: sentry: remove experimental tag from sentry flags .. sentry: remove experimental tag from sentry flags Like the HMS integ

[kudu-CR] hms: remove experimental tag from hms flags

2019-04-26 Thread Andrew Wong (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13124 to look at the new patch set (#3). Change subject: hms: remove experimental tag from hms flags .

[kudu-CR] hms: remove experimental tag from hms flags

2019-04-26 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13124 ) Change subject: hms: remove experimental tag from hms flags .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080

[kudu-CR] [backup] Add more metadata fields

2019-04-26 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13130 ) Change subject: [backup] Add more metadata fields .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/13130/1/java/kudu-ba

[kudu-CR] sentry: remove experimental tag from sentry flags

2019-04-26 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13133 ) Change subject: sentry: remove experimental tag from sentry flags .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.or

[kudu-CR] [master] add RPC to reset AuthzProvider's cache

2019-04-26 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13063 ) Change subject: [master] add RPC to reset AuthzProvider's cache .. Patch Set 6: Code-Review+2 (2 comments) http://gerrit.cloudera.org:8080/

[kudu-CR] sentry: remove experimental tag from sentry flags

2019-04-26 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13133 ) Change subject: sentry: remove experimental tag from sentry flags .. Patch Set 1: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8

[kudu-CR] [master] add RPC to reset AuthzProvider's cache

2019-04-26 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13063 ) Change subject: [master] add RPC to reset AuthzProvider's cache .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.

[kudu-CR] [backup] Ensure the last metadata is restored

2019-04-26 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13132 ) Change subject: [backup] Ensure the last metadata is restored .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/13132/1//

[kudu-CR] hms: remove experimental tag from hms flags

2019-04-26 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13124 ) Change subject: hms: remove experimental tag from hms flags .. Patch Set 3: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/

[kudu-CR] [master] add RPC to reset AuthzProvider's cache

2019-04-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13063 ) Change subject: [master] add RPC to reset AuthzProvider's cache .. [master] add RPC to reset AuthzProvider's cache This change

[kudu-CR] hms: remove experimental tag from hms flags

2019-04-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13124 ) Change subject: hms: remove experimental tag from hms flags .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/13124/3/s

[kudu-CR] sentry: remove experimental tag from sentry flags

2019-04-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13133 ) Change subject: sentry: remove experimental tag from sentry flags .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/131

[kudu-CR] sentry: remove experimental tag from sentry flags

2019-04-26 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13133 ) Change subject: sentry: remove experimental tag from sentry flags .. sentry: remove experimental tag from sentry flags Like the

[kudu-CR] hms: remove experimental tag from hms flags

2019-04-26 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13124 ) Change subject: hms: remove experimental tag from hms flags .. hms: remove experimental tag from hms flags The HMS integration h

[kudu-CR] hms: mark latency flag as hidden

2019-04-26 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13135 Change subject: hms: mark latency flag as hidden .. hms: mark latency flag as hidden Change-Id: Ib11848a262a1d9329a3981c7f7eafb99134fae75

[kudu-CR] hms: remove experimental tag from hms flags

2019-04-26 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13124 ) Change subject: hms: remove experimental tag from hms flags .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/13124/3/src

[kudu-CR] hms: mark latency flag as hidden

2019-04-26 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13135 ) Change subject: hms: mark latency flag as hidden .. Patch Set 1: Retriggered. Looks like the failures are NTP-related. -- To view, vis

[kudu-CR] hms: mark latency flag as hidden

2019-04-26 Thread Andrew Wong (Code Review)
Andrew Wong has removed a vote on this change. Change subject: hms: mark latency flag as hidden .. Removed -Verified by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13135 To unsubscribe, visit http://gerr

[kudu-CR] hms: mark latency flag as hidden

2019-04-26 Thread Andrew Wong (Code Review)
Andrew Wong has removed a vote on this change. Change subject: hms: mark latency flag as hidden .. Removed -Verified by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13135 To unsubscribe, visit http://gerr

[kudu-CR] [master sentry-itest] a bit more concise signature for functions

2019-04-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13137 Change subject: [master_sentry-itest] a bit more concise signature for functions .. [master_sentry-itest] a bit more concise signature f

[kudu-CR] [master sentry-itest] a bit more concise signature for functions

2019-04-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/13137 ) Change subject: [master_sentry-itest] a bit more concise signature for functions .. Removed reviewer Kudu Jenkins with the foll

[kudu-CR] [master sentry-itest] a bit more concise signature for functions

2019-04-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13137 ) Change subject: [master_sentry-itest] a bit more concise signature for functions .. Patch Set 1: Verified+1 Unrelated flakes in: * o

[kudu-CR] [master sentry-itest] a bit more concise signature for functions

2019-04-26 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13137 ) Change subject: [master_sentry-itest] a bit more concise signature for functions .. Patch Set 1: Code-Review+1 -- To view, visit http://

[kudu-CR] hms: mark latency flag as hidden

2019-04-26 Thread Andrew Wong (Code Review)
Andrew Wong has removed a vote on this change. Change subject: hms: mark latency flag as hidden .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13135 To unsubscribe, visit http://ger

[kudu-CR] hms: mark latency flag as hidden

2019-04-26 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13135 ) Change subject: hms: mark latency flag as hidden .. Patch Set 1: Verified+1 Seems like a test failed due to NTP issues. It's unrelated.

[kudu-CR] WIP [master sentry-itest] a couple of yet disabled tests

2019-04-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13138 Change subject: WIP [master_sentry-itest] a couple of yet disabled tests .. WIP [master_sentry-itest] a couple of yet disabled tests Ad

[kudu-CR] hms: mark latency flag as hidden

2019-04-26 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13135 ) Change subject: hms: mark latency flag as hidden .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/13135 T

[kudu-CR] hms: mark latency flag as hidden

2019-04-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13135 ) Change subject: hms: mark latency flag as hidden .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/1313

[kudu-CR] WIP [master sentry-itest] a couple of yet disabled tests

2019-04-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/13138 ) Change subject: WIP [master_sentry-itest] a couple of yet disabled tests .. Removed reviewer Kudu Jenkins with the following vo

[kudu-CR] WIP [master sentry-itest] a couple of yet disabled tests

2019-04-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13138 ) Change subject: WIP [master_sentry-itest] a couple of yet disabled tests .. Patch Set 1: Verified+1 unrelated flakes: * org.apache.k

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

2019-04-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13139 Change subject: [tools] command to reset authz cache in kudu CLI .. [tools] command to reset authz cache in kudu CLI Added command to r

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

2019-04-26 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 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/13139

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

2019-04-26 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 (#2). Change subject: [tools] command to reset authz cache in kudu CLI ..

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

2019-04-26 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 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/131