[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13510 ) Change subject: hms: allow for tooling to run without Kudu plugin .. Patch Set 5: Code-Review+2 -- To view, visit

[kudu-CR] [hms] Adjust storage handler package

2019-06-06 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13540 ) Change subject: [hms] Adjust storage handler package .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/13540/1//COMMIT_MSG

[kudu-CR] [hms] disallow table type altering via table property

2019-06-05 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13524 ) Change subject: [hms] disallow table type altering via table property .. Patch Set 2: (3 comments)

[kudu-CR] [hms] disallow table type altering via table property

2019-06-05 Thread Hao Hao (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13524 to look at the new patch set (#2). Change subject: [hms] disallow table type altering via table property

[kudu-CR] sentry: don't send requests for DATABASE/SERVER privileges

2019-06-05 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13494 ) Change subject: sentry: don't send requests for DATABASE/SERVER privileges .. Patch Set 4: Code-Review+2 -- To view, visit

[kudu-CR] [hms] disallow table type altering via table property

2019-06-05 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13524 Change subject: [hms] disallow table type altering via table property .. [hms] disallow table type altering via table property This blocks

[kudu-CR] sentry: add trace points

2019-06-04 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13493 ) Change subject: sentry: add trace points .. sentry: add trace points Adds TRACE statements at various points in Sentry

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13510 ) Change subject: hms: allow for tooling to run without Kudu plugin .. Patch Set 2: (5 comments)

[kudu-CR] sentry: add trace points

2019-06-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13493 ) Change subject: sentry: add trace points .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/13493 To

[kudu-CR] [hms] Support repairing entries with invalid IDs

2019-06-03 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13492 ) Change subject: [hms] Support repairing entries with invalid IDs .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] sentry: don't send requests for DATABASE/SERVER privileges

2019-06-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13494 ) Change subject: sentry: don't send requests for DATABASE/SERVER privileges .. Patch Set 2: Code-Review+1 (2 comments) LGTM, just one minor

[kudu-CR] sentry: add trace points

2019-06-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13493 ) Change subject: sentry: add trace points .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/13493/1//COMMIT_MSG Commit

[kudu-CR] [hms] Support repairing entries with invalid IDs

2019-06-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13492 ) Change subject: [hms] Support repairing entries with invalid IDs .. Patch Set 2: Code-Review+1 (1 comment) LGTM, just one minor issue.

[kudu-CR] hms: have tools ignore other clusters

2019-05-30 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13411 ) Change subject: hms: have tools ignore other clusters .. Patch Set 5: Code-Review+2 (1 comment)

[kudu-CR] hms: have tools ignore other clusters

2019-05-30 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13411 ) Change subject: hms: have tools ignore other clusters .. hms: have tools ignore other clusters Currently, the HMS tooling will

[kudu-CR] hms: skip drop table notification log for external table

2019-05-28 Thread Hao Hao (Code Review)
Hao Hao has abandoned this change. ( http://gerrit.cloudera.org:8080/13073 ) Change subject: hms: skip drop table notification log for external table .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/13073 To

[kudu-CR] hms: have tools ignore other clusters

2019-05-28 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13411 ) Change subject: hms: have tools ignore other clusters .. Patch Set 4: (2 comments)

[kudu-CR] hms plugin: don't expect table type

2019-05-24 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13424 ) Change subject: hms plugin: don't expect table type .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/13424

[kudu-CR] java: incorporate feedback from commit 37e53bf1d

2019-05-24 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13423 ) Change subject: java: incorporate feedback from commit 37e53bf1d .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] java: incorporate feedback from commit 37e53bf1d

2019-05-24 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13423 ) Change subject: java: incorporate feedback from commit 37e53bf1d .. java: incorporate feedback from commit 37e53bf1d Change-Id:

[kudu-CR] hms: allow skip validation in Hive Metastore Kudu plugin

2019-05-23 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: hms: allow skip validation in Hive Metastore Kudu plugin .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13387 To

[kudu-CR] hms: allow skip validation in Hive Metastore Kudu plugin

2019-05-23 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13387 ) Change subject: hms: allow skip validation in Hive Metastore Kudu plugin .. hms: allow skip validation in Hive Metastore Kudu plugin

[kudu-CR] hms: fix flakiness of master-stress-test from managed tables

2019-05-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13384 ) Change subject: hms: fix flakiness of master-stress-test from managed tables .. Patch Set 3: Code-Review+2 (2 comments)

[kudu-CR] hms: allow skip validation in Hive Metastore Kudu plugin

2019-05-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13387 ) Change subject: hms: allow skip validation in Hive Metastore Kudu plugin .. Patch Set 2: Unrelated test failure. -- To view, visit

[kudu-CR] hms: allow skip validation in Hive Metastore Kudu plugin

2019-05-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13387 ) Change subject: hms: allow skip validation in Hive Metastore Kudu plugin .. Patch Set 2: Verified+1 (3 comments)

[kudu-CR] hms: followup to 6a8d1d2

2019-05-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13410 ) Change subject: hms: followup to 6a8d1d2 .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/13410 To

[kudu-CR] hms: fix flakiness of master-stress-test from managed tables

2019-05-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13384 ) Change subject: hms: fix flakiness of master-stress-test from managed tables .. Patch Set 2: Code-Review+1 (2 comments)

[kudu-CR] java: reduce root loglevel back to INFO

2019-05-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13417 ) Change subject: java: reduce root loglevel back to INFO .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] java: fix CapturingToFileLogAppender

2019-05-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13418 ) Change subject: java: fix CapturingToFileLogAppender .. Patch Set 1: Code-Review+2 (1 comment)

[kudu-CR] hms: allow skip validation in Hive Metastore Kudu plugin

2019-05-23 Thread Hao Hao (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13387 to look at the new patch set (#2). Change subject: hms: allow skip validation in Hive Metastore Kudu plugin

[kudu-CR] hms: followup to 6a8d1d2

2019-05-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13410 ) Change subject: hms: followup to 6a8d1d2 .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/13410 To

[kudu-CR] hms: have tools ignore other clusters

2019-05-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13411 ) Change subject: hms: have tools ignore other clusters .. Patch Set 2: (1 comment)

[kudu-CR] hms: followup to 6a8d1d2

2019-05-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13410 ) Change subject: hms: followup to 6a8d1d2 .. Patch Set 3: Code-Review+2 (1 comment)

[kudu-CR] Allow alter legacy tables in Hive Metastore Kudu plugin

2019-05-22 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13317 ) Change subject: Allow alter legacy tables in Hive Metastore Kudu plugin .. Allow alter legacy tables in Hive Metastore Kudu plugin

[kudu-CR] hms: allow skip validation in Hive Metastore Kudu plugin

2019-05-21 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13387 ) Change subject: hms: allow skip validation in Hive Metastore Kudu plugin .. Patch Set 1: > It can be in a follow up, but can you add a test

[kudu-CR] hms: allow skip validation in Hive Metastore Kudu plugin

2019-05-21 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13387 ) Change subject: hms: allow skip validation in Hive Metastore Kudu plugin .. Patch Set 1: Verified+1 Unrelated flaky test. -- To view,

[kudu-CR] hms: allow skip validation in Hive Metastore Kudu plugin

2019-05-21 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: hms: allow skip validation in Hive Metastore Kudu plugin .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13387 To

[kudu-CR] hms: synchronize column comments to the HMS

2019-05-21 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13381 ) Change subject: hms: synchronize column comments to the HMS .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] Allow alter legacy tables in Hive Metastore Kudu plugin

2019-05-21 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13317 ) Change subject: Allow alter legacy tables in Hive Metastore Kudu plugin .. Patch Set 6: (1 comment)

[kudu-CR] hms: allow skip validation in Hive Metastore Kudu plugin

2019-05-21 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13387 Change subject: hms: allow skip validation in Hive Metastore Kudu plugin .. hms: allow skip validation in Hive Metastore Kudu plugin This

[kudu-CR] Allow alter legacy tables in Hive Metastore Kudu plugin

2019-05-17 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13317 ) Change subject: Allow alter legacy tables in Hive Metastore Kudu plugin .. Patch Set 5: (7 comments)

[kudu-CR] Allow alter legacy tables in Hive Metastore Kudu plugin

2019-05-17 Thread Hao Hao (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13317 to look at the new patch set (#6). Change subject: Allow alter legacy tables in Hive Metastore Kudu plugin

[kudu-CR] Allow alter legacy tables in Hive Metastore Kudu plugin

2019-05-16 Thread Hao Hao (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13317 to look at the new patch set (#5). Change subject: Allow alter legacy tables in Hive Metastore Kudu plugin

[kudu-CR] Allow alter legacy tables in Hive Metastore Kudu plugin

2019-05-16 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13317 ) Change subject: Allow alter legacy tables in Hive Metastore Kudu plugin .. Patch Set 4: (3 comments)

[kudu-CR] authz: mark --trusted user acl as non-experimental

2019-05-16 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13351 ) Change subject: authz: mark --trusted_user_acl as non-experimental .. authz: mark --trusted_user_acl as non-experimental This is a

[kudu-CR] authz: mark --trusted user acl as non-experimental

2019-05-16 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13351 ) Change subject: authz: mark --trusted_user_acl as non-experimental .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] Allow alter legacy tables in Hive Metastore Kudu plugin

2019-05-14 Thread Hao Hao (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13317 to look at the new patch set (#4). Change subject: Allow alter legacy tables in Hive Metastore Kudu plugin

[kudu-CR] Allow alter legacy tables in Hive Metastore Kudu plugin

2019-05-14 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: Allow alter legacy tables in Hive Metastore Kudu plugin .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13317 To

[kudu-CR] Allow alter legacy tables in Hive Metastore Kudu plugin

2019-05-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13317 ) Change subject: Allow alter legacy tables in Hive Metastore Kudu plugin .. Patch Set 3: Verified+1 Unrelated flaky test failure. -- To

[kudu-CR] Allow alter legacy tables in Hive Metastore Kudu plugin

2019-05-14 Thread Hao Hao (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13317 to look at the new patch set (#3). Change subject: Allow alter legacy tables in Hive Metastore Kudu plugin

[kudu-CR] Allow alter legacy tables in Hive Metastore Kudu plugin

2019-05-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13317 ) Change subject: Allow alter legacy tables in Hive Metastore Kudu plugin .. Patch Set 3: (1 comment)

[kudu-CR] Allow alter legacy tables in Hive Metastore Kudu plugin

2019-05-13 Thread Hao Hao (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13317 to look at the new patch set (#2). Change subject: Allow alter legacy tables in Hive Metastore Kudu plugin

[kudu-CR] [master sentry-itest] one more scenario for authz cache

2019-05-13 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13291 ) Change subject: [master_sentry-itest] one more scenario for authz cache .. Patch Set 4: Code-Review+2 (2 comments)

[kudu-CR] Allow alter legacy tables in Hive Metastore Kudu plugin

2019-05-13 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13317 ) Change subject: Allow alter legacy tables in Hive Metastore Kudu plugin .. Patch Set 2: (7 comments)

[kudu-CR] [gradle] publish kudu-hive artifact

2019-05-13 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13316 ) Change subject: [gradle] publish kudu-hive artifact .. [gradle] publish kudu-hive artifact This patch allows publishing kudu-hive

[kudu-CR] [master sentry-itest] one more scenario for authz cache

2019-05-13 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13291 ) Change subject: [master_sentry-itest] one more scenario for authz cache .. Patch Set 3: (3 comments)

[kudu-CR] Allow alter legacy tables in Hive Metastore Kudu plugin

2019-05-13 Thread Hao Hao (Code Review)
Hao Hao has removed Tony Foerster from this change. ( http://gerrit.cloudera.org:8080/13317 ) Change subject: Allow alter legacy tables in Hive Metastore Kudu plugin .. Removed reviewer Tony Foerster. -- To view, visit

[kudu-CR] [gradle] publish kudu-hive artifact

2019-05-13 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13316 Change subject: [gradle] publish kudu-hive artifact .. [gradle] publish kudu-hive artifact This patch allows publishing kudu-hive artifact,

[kudu-CR] Allow alter legacy tables in Hive Metastore Kudu plugin

2019-05-13 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13317 Change subject: Allow alter legacy tables in Hive Metastore Kudu plugin .. Allow alter legacy tables in Hive Metastore Kudu plugin

[kudu-CR] WIP [java] Upgrade Hive dependency to 3.1.1

2019-05-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13256 ) Change subject: WIP [java] Upgrade Hive dependency to 3.1.1 .. Patch Set 2: (1 comment) > Hao should take a look at this. > > In general

[kudu-CR] [catalog manager] run sanity checks before catching up with HMS

2019-05-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13257 ) Change subject: [catalog_manager] run sanity checks before catching up with HMS .. Patch Set 3: (1 comment)

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

2019-05-06 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 7: Code-Review+2 -- To view, visit

[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)

[kudu-CR] [TTL cache] add periodic scrubbing thread

2019-05-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13201 ) Change subject: [TTL cache] add periodic scrubbing thread .. Patch Set 8: Code-Review+2 -- To view, visit

[kudu-CR] [TTL cache] add periodic scrubbing thread

2019-05-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13201 ) Change subject: [TTL cache] add periodic scrubbing thread .. Patch Set 7: Code-Review+1 (2 comments) LGTM, just some nits

[kudu-CR] [authz] add scrubbing task for the privileges cache

2019-05-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13205 ) Change subject: [authz] add scrubbing task for the privileges cache .. Patch Set 5: Code-Review+2 -- To view, visit

[kudu-CR] Normalize sentry package name

2019-05-01 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13203 ) Change subject: Normalize sentry package name .. Normalize sentry package name This strips the 'apache' prefix and 'bin' suffix

[kudu-CR] Normalize sentry package name

2019-04-30 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13203 ) Change subject: Normalize sentry package name .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/13203/1//COMMIT_MSG Commit

[kudu-CR] Normalize sentry package name

2019-04-30 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: Normalize sentry package name .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13203 To unsubscribe, visit

[kudu-CR] Normalize sentry package name

2019-04-30 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13203 ) Change subject: Normalize sentry package name .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/13203 To

[kudu-CR] Normalize sentry package name

2019-04-30 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13203 ) Change subject: Normalize sentry package name .. Patch Set 1: Unrelated java test failure. -- To view, visit

[kudu-CR] Normalize sentry package name

2019-04-30 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13203 Change subject: Normalize sentry package name .. Normalize sentry package name This strips the 'apache' prefix and 'bin' suffix from the

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

2019-04-29 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 3: Code-Review+2 (1 comment) May want to wait for Andrew or

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

2019-04-28 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 2: (2 comments)

[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)

[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

[kudu-CR] Make HiveMetastoreConfig class public

2019-04-26 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13128 Change subject: Make HiveMetastoreConfig class public .. Make HiveMetastoreConfig class public Commit 2c572cbd4 introduced the Java client

[kudu-CR] [authz] more tests for SentryPrivilegesFetcher's cache

2019-04-25 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13125 ) Change subject: [authz] more tests for SentryPrivilegesFetcher's cache .. Patch Set 1: Code-Review+2 -- To view, visit

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

2019-04-25 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 5: Code-Review+1 (6 comments) Just few nits.

[kudu-CR] [authz] new SentryAuthzProvider's caching strategy

2019-04-25 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13069 ) Change subject: [authz] new SentryAuthzProvider's caching strategy .. Patch Set 9: Code-Review+2 (1 comment) LGTM, just one minor nit.

[kudu-CR] [authz] new SentryAuthzProvider's caching strategy

2019-04-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13069 ) Change subject: [authz] new SentryAuthzProvider's caching strategy .. Patch Set 6: Code-Review+1 (5 comments) LGTM, just a few nits.

[kudu-CR] [sentry] enable sentry integration for master stress test

2019-04-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12877 ) Change subject: [sentry] enable sentry integration for master stress test .. Patch Set 5: Verified+1 Flaky test

[kudu-CR] [sentry] enable sentry integration for master stress test

2019-04-23 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12877 ) Change subject: [sentry] enable sentry integration for master stress test .. [sentry] enable sentry integration for master stress

[kudu-CR] [authz] new SentryAuthzProvider's caching strategy

2019-04-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13069 ) Change subject: [authz] new SentryAuthzProvider's caching strategy .. Patch Set 5: (2 comments)

[kudu-CR] [sentry] enable sentry integration for master stress test

2019-04-23 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: [sentry] enable sentry integration for master stress test .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/12877 To

[kudu-CR] [sentry] enable sentry integration for master stress test

2019-04-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12877 ) Change subject: [sentry] enable sentry integration for master stress test .. Patch Set 4: (2 comments)

[kudu-CR] [sentry] enable sentry integration for master stress test

2019-04-23 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12877 to look at the new patch set (#5). Change subject: [sentry] enable sentry integration for master stress test

[kudu-CR] master: use AuthzProvider to generate authz tokens

2019-04-22 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13013 ) Change subject: master: use AuthzProvider to generate authz tokens .. Patch Set 9: Code-Review+2 (2 comments)

[kudu-CR] [sentry] enable sentry integration for master stress test

2019-04-22 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12877 ) Change subject: [sentry] enable sentry integration for master stress test .. Patch Set 3: (4 comments)

[kudu-CR] [sentry] enable sentry integration for master stress test

2019-04-22 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12877 to look at the new patch set (#4). Change subject: [sentry] enable sentry integration for master stress test

[kudu-CR] [sentry] enable sentry integration for master stress test

2019-04-21 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12877 to look at the new patch set (#3). Change subject: [sentry] enable sentry integration for master stress test

[kudu-CR] [sentry] enable sentry integration for master stress test

2019-04-19 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12877 ) Change subject: [sentry] enable sentry integration for master stress test .. Patch Set 2: Messed up the latest patch, fixing the test

[kudu-CR] [sentry] enable sentry integration for master stress test

2019-04-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12877 ) Change subject: [sentry] enable sentry integration for master stress test .. Patch Set 2: (16 comments)

[kudu-CR] WIP master: use AuthzProvider to generate authz tokens

2019-04-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13013 ) Change subject: WIP master: use AuthzProvider to generate authz tokens .. Patch Set 4: (1 comment)

[kudu-CR] [sentry] enable sentry integration for master stress test

2019-04-18 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12877 to look at the new patch set (#2). Change subject: [sentry] enable sentry integration for master stress test

[kudu-CR] Bump Sentry version

2019-04-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13059 ) Change subject: Bump Sentry version .. Patch Set 2: Verified+1 Unrelated tsan failure. -- To view, visit

[kudu-CR] Bump Sentry version

2019-04-18 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: Bump Sentry version .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13059 To unsubscribe, visit

[kudu-CR] Bump Sentry version

2019-04-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13059 ) Change subject: Bump Sentry version .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/13059/2/thirdparty/build-thirdparty.sh

[kudu-CR] Bump Sentry version

2019-04-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13059 ) Change subject: Bump Sentry version .. Patch Set 1: (8 comments) http://gerrit.cloudera.org:8080/#/c/13059/1//COMMIT_MSG Commit Message:

[kudu-CR] Bump Sentry version

2019-04-18 Thread Hao Hao (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13059 to look at the new patch set (#2). Change subject: Bump Sentry version ..

<    3   4   5   6   7   8   9   10   11   12   >