[kudu-CR] [java] deflake RYW tests in TestKuduClient

2019-01-28 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12276 ) Change subject: [java] deflake RYW tests in TestKuduClient .. [java] deflake RYW tests in TestKuduClient RYW tests in

[kudu-CR] [java] deflake RYW tests in TestKuduClient

2019-01-28 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12276 to look at the new patch set (#4). Change subject: [java] deflake RYW tests in TestKuduClient ..

[kudu-CR] [java] deflake RYW tests in TestKuduClient

2019-01-28 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12276 ) Change subject: [java] deflake RYW tests in TestKuduClient .. Patch Set 4: (3 comments)

[kudu-CR] [java] enable regression test for KUDU-2415

2019-01-28 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12278 ) Change subject: [java] enable regression test for KUDU-2415 .. [java] enable regression test for KUDU-2415 As KUDU-2463 fixed the

[kudu-CR] [java] enable regression test for KUDU-2415

2019-01-28 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12278 ) Change subject: [java] enable regression test for KUDU-2415 .. Patch Set 2: Code-Review+2 Carry Andrew's +2 from last patch -- To view,

[kudu-CR] [java] deflake RYW tests in TestKuduClient

2019-01-28 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: [java] deflake RYW tests in TestKuduClient .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/12276 To unsubscribe, visit

[kudu-CR] [java] deflake RYW tests in TestKuduClient

2019-01-28 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12276 ) Change subject: [java] deflake RYW tests in TestKuduClient .. Patch Set 3: Verified+1 Unrelated flaky test. -- To view, visit

[kudu-CR] [java] deflake RYW tests in TestKuduClient

2019-01-27 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12276 to look at the new patch set (#3). Change subject: [java] deflake RYW tests in TestKuduClient ..

[kudu-CR] [java] deflake RYW tests in TestKuduClient

2019-01-27 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12276 to look at the new patch set (#2). Change subject: [java] deflake RYW tests in TestKuduClient ..

[kudu-CR] [java] enable regression test for KUDU-2415

2019-01-27 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12278 to look at the new patch set (#2). Change subject: [java] enable regression test for KUDU-2415 ..

[kudu-CR] [java] enable regression test for KUDU-2415

2019-01-27 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12278 ) Change subject: [java] enable regression test for KUDU-2415 .. Patch Set 2: (1 comment)

[kudu-CR] [java] enable regression test for KUDU-2415

2019-01-26 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12278 ) Change subject: [java] enable regression test for KUDU-2415 .. Patch Set 1: (1 comment)

[kudu-CR] [java] enable regression test for KUDU-2415

2019-01-25 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12278 Change subject: [java] enable regression test for KUDU-2415 .. [java] enable regression test for KUDU-2415 As KUDU-2463 fixed the bug in

[kudu-CR] [java] deflake RYW tests in TestKuduClient

2019-01-25 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12276 Change subject: [java] deflake RYW tests in TestKuduClient .. [java] deflake RYW tests in TestKuduClient RYW tests in TestKuduClient is

[kudu-CR] Fix master sentry-itest built in RELEASE mode

2019-01-25 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12274 ) Change subject: Fix master_sentry-itest built in RELEASE mode .. Fix master_sentry-itest built in RELEASE mode Commit 9098f442e

[kudu-CR] Fix master sentry-itest built in RELEASE mode

2019-01-25 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12274 to look at the new patch set (#2). Change subject: Fix master_sentry-itest built in RELEASE mode ..

[kudu-CR] Fix master sentry-itest built in RELEASE mode

2019-01-24 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12274 ) Change subject: Fix master_sentry-itest built in RELEASE mode .. Patch Set 1: (1 comment) > (1 comment) > > > It is necessary to not

[kudu-CR] Fix master sentry-itest built in RELEASE mode

2019-01-24 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12274 ) Change subject: Fix master_sentry-itest built in RELEASE mode .. Patch Set 1: (1 comment) > (1 comment) > > The changes to

[kudu-CR] [sentry] Integrate AuthzProvider into CatalogManager

2019-01-24 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 3: (2 comments)

[kudu-CR] Fix master sentry-itest built in RELEASE mode

2019-01-24 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12274 Change subject: Fix master_sentry-itest built in RELEASE mode .. Fix master_sentry-itest built in RELEASE mode Commit 9098f442e introduced a

[kudu-CR] [fs]: delete the (orphaned) metadata file when repairing

2019-01-24 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12239 ) Change subject: [fs]: delete the (orphaned) metadata file when repairing .. Patch Set 8: (1 comment) LGTM. @Adar, I can merge it if you

[kudu-CR] [fs]: delete the (orphaned) metadata file when repairing

2019-01-24 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12239 ) Change subject: [fs]: delete the (orphaned) metadata file when repairing .. Patch Set 7: (1 comment)

[kudu-CR] [fs]: delete the (orphaned) metadata file when repairing

2019-01-24 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12239 ) Change subject: [fs]: delete the (orphaned) metadata file when repairing .. Patch Set 8: Rebase the patch to pick up the python fix. --

[kudu-CR] [fs]: delete the (orphaned) metadata file when repairing

2019-01-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12239 ) Change subject: [fs]: delete the (orphaned) metadata file when repairing .. Patch Set 6: (5 comments)

[kudu-CR] De-flake sentry tests

2019-01-23 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11961 ) Change subject: De-flake sentry tests .. De-flake sentry tests Currently, sentry_client-test and sentry_authz_provider-test are

[kudu-CR] KUDU-2543 pt 2: pass around default authz tokens

2019-01-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12122 ) Change subject: KUDU-2543 pt 2: pass around default authz tokens .. Patch Set 9: Code-Review+2 (1 comment)

[kudu-CR] KUDU-2543 pt 1: basic checks for authz tokens

2019-01-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11751 ) Change subject: KUDU-2543 pt 1: basic checks for authz tokens .. Patch Set 12: Code-Review+2 -- To view, visit

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2019-01-23 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11956 ) Change subject: [sentry] add mini Sentry to the external mini cluster .. [sentry] add mini Sentry to the external mini cluster This

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2019-01-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11956 ) Change subject: [sentry] add mini Sentry to the external mini cluster .. Patch Set 12: (2 comments)

[kudu-CR] KUDU-2543 pt 2: pass around default authz tokens

2019-01-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12122 ) Change subject: KUDU-2543 pt 2: pass around default authz tokens .. Patch Set 6: Code-Review+1 (1 comment)

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2019-01-23 Thread Hao Hao (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11956 to look at the new patch set (#12). Change subject: [sentry] add mini Sentry to the external mini cluster

[kudu-CR] KUDU-2543 pt 1: basic checks for authz tokens

2019-01-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11751 ) Change subject: KUDU-2543 pt 1: basic checks for authz tokens .. Patch Set 11: Code-Review+2 -- To view, visit

[kudu-CR] [java] log warning if applying operation on a closed session

2019-01-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12237 ) Change subject: [java] log warning if applying operation on a closed session .. Patch Set 5: Code-Review+2 -- To view, visit

[kudu-CR] python: fix Python 3.4 based tests

2019-01-22 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12255 Change subject: python: fix Python 3.4 based tests .. python: fix Python 3.4 based tests Pip released 19.0 recently and tests based on

[kudu-CR] De-flake sentry tests

2019-01-22 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11961 to look at the new patch set (#6). Change subject: De-flake sentry tests .. De-flake

[kudu-CR] De-flake sentry tests

2019-01-22 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11961 ) Change subject: De-flake sentry tests .. Patch Set 5: (2 comments)

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2019-01-22 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11956 ) Change subject: [sentry] add mini Sentry to the external mini cluster .. Patch Set 11: (5 comments)

[kudu-CR] De-flake sentry tests

2019-01-22 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11961 to look at the new patch set (#5). Change subject: De-flake sentry tests .. De-flake

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2019-01-22 Thread Hao Hao (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11956 to look at the new patch set (#11). Change subject: [sentry] add mini Sentry to the external mini cluster

[kudu-CR] [java] throw exception if applying operation on a closed session

2019-01-17 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12237 ) Change subject: [java] throw exception if applying operation on a closed session .. Patch Set 4: > Like you said, this is a breaking change

[kudu-CR] KUDU-2636: LBM supports deleting dead containers

2019-01-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead containers .. Patch Set 16: Code-Review+1 (2 comments) Just some minor nits.

[kudu-CR] [examples] fix name of the class for spark-submit

2019-01-10 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12206 ) Change subject: [examples] fix name of the class for spark-submit .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [examples] fix name of the class for spark-submit

2019-01-09 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12206 ) Change subject: [examples] fix name of the class for spark-submit .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2019-01-09 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead and full containers .. Patch Set 12: (6 comments)

[kudu-CR] KUDU-2543 pt 2: pass around default authz tokens

2019-01-08 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12122 ) Change subject: KUDU-2543 pt 2: pass around default authz tokens .. Patch Set 5: (12 comments)

[kudu-CR] KUDU-2543 pt 1: basic checks for authz tokens

2019-01-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11751 ) Change subject: KUDU-2543 pt 1: basic checks for authz tokens .. Patch Set 10: (3 comments) LGTM, just minor nits.

[kudu-CR] [docs] add Hive Metastore integration

2019-01-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11798 ) Change subject: [docs] add Hive Metastore integration .. Patch Set 8: Code-Review+2 -- To view, visit

[kudu-CR] [gradle] Increase test JVMs heap size

2018-12-27 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12134 Change subject: [gradle] Increase test JVMs heap size .. [gradle] Increase test JVMs heap size The flume test

[kudu-CR] [docs] add Hive Metastore integration

2018-12-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11798 ) Change subject: [docs] add Hive Metastore integration .. Patch Set 7: (8 comments)

[kudu-CR] [docs] add Hive Metastore integration

2018-12-14 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new patch set (#7) to the change originally created by Dan Burkert. ( http://gerrit.cloudera.org:8080/11798 ) Change subject: [docs] add Hive Metastore integration .. [docs] add Hive Metastore integration

[kudu-CR] KUDU-683: unify C++ client-to-master retry logic

2018-12-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12002 ) Change subject: KUDU-683: unify C++ client-to-master retry logic .. Patch Set 5: (5 comments)

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2018-12-13 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11956 ) Change subject: [sentry] add mini Sentry to the external mini cluster .. Patch Set 9: (2 comments)

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2018-12-13 Thread Hao Hao (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11956 to look at the new patch set (#10). Change subject: [sentry] add mini Sentry to the external mini cluster

[kudu-CR] De-flake sentry tests

2018-12-12 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11961 ) Change subject: De-flake sentry tests .. Patch Set 4: (5 comments)

[kudu-CR] De-flake sentry tests

2018-12-12 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11961 to look at the new patch set (#4). Change subject: De-flake sentry tests .. De-flake

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2018-12-12 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11956 ) Change subject: [sentry] add mini Sentry to the external mini cluster .. Patch Set 9: (1 comment)

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2018-12-12 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11956 ) Change subject: [sentry] add mini Sentry to the external mini cluster .. Patch Set 9: (17 comments)

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2018-12-12 Thread Hao Hao (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11956 to look at the new patch set (#9). Change subject: [sentry] add mini Sentry to the external mini cluster

[kudu-CR] [docs] add Hive Metastore integration

2018-12-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11798 ) Change subject: [docs] add Hive Metastore integration .. Patch Set 6: (24 comments)

[kudu-CR] [docs] add Hive Metastore integration

2018-12-11 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new patch set (#6) to the change originally created by Dan Burkert. ( http://gerrit.cloudera.org:8080/11798 ) Change subject: [docs] add Hive Metastore integration .. [docs] add Hive Metastore integration

[kudu-CR] De-flake sentry tests

2018-12-10 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11961 to look at the new patch set (#3). Change subject: De-flake sentry tests .. De-flake

[kudu-CR] De-flake sentry tests

2018-12-10 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11961 ) Change subject: De-flake sentry tests .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/11961/1/src/kudu/util/test_util.h

[kudu-CR] De-flake sentry tests

2018-12-10 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11961 to look at the new patch set (#2). Change subject: De-flake sentry tests .. De-flake

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2018-12-09 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11956 ) Change subject: [sentry] add mini Sentry to the external mini cluster .. Patch Set 3: (11 comments)

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2018-12-09 Thread Hao Hao (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11956 to look at the new patch set (#8). Change subject: [sentry] add mini Sentry to the external mini cluster

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2018-12-09 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11956 ) Change subject: [sentry] add mini Sentry to the external mini cluster .. Patch Set 7: (1 comment)

[kudu-CR] [sentry] Integrate AuthzProvider into CatalogManager

2018-12-09 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 2: (31 comments)

[kudu-CR] [sentry] Integrate AuthzProvider into CatalogManager

2018-12-09 Thread Hao Hao (Code Review)
Hello Tidy Bot, Dan Burkert, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11797 to look at the new patch set (#3). Change subject: [sentry] Integrate AuthzProvider into CatalogManager

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2018-12-09 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11956 ) Change subject: [sentry] add mini Sentry to the external mini cluster .. Patch Set 7: (42 comments)

[kudu-CR] [sentry] Integrate AuthzProvider into CatalogManager

2018-12-09 Thread Hao Hao (Code Review)
Hao Hao has abandoned this change. ( http://gerrit.cloudera.org:8080/12058 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/12058 To

[kudu-CR] [sentry] Integrate AuthzProvider into CatalogManager

2018-12-09 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12058 Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. [sentry] Integrate AuthzProvider into CatalogManager This commit

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2018-12-09 Thread Hao Hao (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11956 to look at the new patch set (#7). Change subject: [sentry] add mini Sentry to the external mini cluster

[kudu-CR] WIP:[sentry] Integrate AuthzProvider into CatalogManager

2018-12-04 Thread Hao Hao (Code Review)
Hello Dan Burkert, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11797 to look at the new patch set (#2). Change subject: WIP:[sentry] Integrate AuthzProvider into CatalogManager

[kudu-CR] [docs] add Hive Metastore integration

2018-11-20 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11798 ) Change subject: [docs] add Hive Metastore integration .. Patch Set 5: (1 comment)

[kudu-CR] [docs] add Hive Metastore integration

2018-11-20 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new patch set (#5) to the change originally created by Dan Burkert. ( http://gerrit.cloudera.org:8080/11798 ) Change subject: [docs] add Hive Metastore integration .. [docs] add Hive Metastore integration

[kudu-CR] [docs] add Hive Metastore integration

2018-11-20 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11798 ) Change subject: [docs] add Hive Metastore integration .. Patch Set 4: (1 comment)

[kudu-CR] [docs] add Hive Metastore integration

2018-11-20 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new patch set (#4) to the change originally created by Dan Burkert. ( http://gerrit.cloudera.org:8080/11798 ) Change subject: [docs] add Hive Metastore integration .. [docs] add Hive Metastore integration

[kudu-CR] De-flake sentry tests

2018-11-20 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11961 ) Change subject: De-flake sentry tests .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/11961/1/src/kudu/util/test_util.cc

[kudu-CR] [docs] add Hive Metastore integration

2018-11-20 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new patch set (#3) to the change originally created by Dan Burkert. ( http://gerrit.cloudera.org:8080/11798 ) Change subject: [docs] add Hive Metastore integration .. [docs] add Hive Metastore integration

[kudu-CR] [docs] add Hive Metastore integration

2018-11-20 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11798 ) Change subject: [docs] add Hive Metastore integration .. Patch Set 2: (6 comments)

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2018-11-20 Thread Hao Hao (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11956 to look at the new patch set (#5). Change subject: [sentry] add mini Sentry to the external mini cluster

[kudu-CR] De-flake sentry tests

2018-11-20 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11961 ) Change subject: De-flake sentry tests .. Patch Set 1: Verified+1 Unrelated flaky ConcurrentRebalancersTest. -- To view, visit

[kudu-CR] De-flake sentry tests

2018-11-20 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: De-flake sentry tests .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/11961 To unsubscribe, visit

[kudu-CR] De-flake sentry tests

2018-11-19 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11961 Change subject: De-flake sentry tests .. De-flake sentry tests Currently, sentry_client-test and sentry_authz_provider-test are still flaky

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2018-11-19 Thread Hao Hao (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11956 to look at the new patch set (#4). Change subject: [sentry] add mini Sentry to the external mini cluster

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2018-11-19 Thread Hao Hao (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11956 to look at the new patch set (#3). Change subject: [sentry] add mini Sentry to the external mini cluster

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2018-11-19 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: [sentry] add mini Sentry to the external mini cluster .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/11956 To

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2018-11-19 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11956 ) Change subject: [sentry] add mini Sentry to the external mini cluster .. Patch Set 2: Verified+1 Unrelated flaky TraceEventCallbackTest.

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2018-11-19 Thread Hao Hao (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11956 to look at the new patch set (#2). Change subject: [sentry] add mini Sentry to the external mini cluster

[kudu-CR] [docs] add Hive Metastore integration

2018-11-19 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new patch set (#2) to the change originally created by Dan Burkert. ( http://gerrit.cloudera.org:8080/11798 ) Change subject: [docs] add Hive Metastore integration .. [docs] add Hive Metastore integration

[kudu-CR] [sentry] add mini Sentry to the external mini cluster

2018-11-19 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11956 Change subject: [sentry] add mini Sentry to the external mini cluster .. [sentry] add mini Sentry to the external mini cluster This commit

[kudu-CR] Fix some issues in HMS and Sentry test fixtures

2018-11-16 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11943 ) Change subject: Fix some issues in HMS and Sentry test fixtures .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [kudu-jepsen] fixed typo in README.adoc

2018-11-14 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11932 ) Change subject: [kudu-jepsen] fixed typo in README.adoc .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [java] Upgrade to Spark 2.4

2018-11-08 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11912 ) Change subject: [java] Upgrade to Spark 2.4 .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/11912 To

[kudu-CR] De-flake sentry client-test and sentry authz provider-test

2018-11-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11898 ) Change subject: De-flake sentry_client-test and sentry_authz_provider-test .. Patch Set 2: (1 comment)

[kudu-CR] De-flake sentry client-test and sentry authz provider-test

2018-11-07 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11898 ) Change subject: De-flake sentry_client-test and sentry_authz_provider-test .. De-flake sentry_client-test and

[kudu-CR] De-flake sentry client-test and sentry authz provider-test

2018-11-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11898 ) Change subject: De-flake sentry_client-test and sentry_authz_provider-test .. Patch Set 1: (1 comment)

[kudu-CR] De-flake sentry client-test and sentry authz provider-test

2018-11-06 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11898 ) Change subject: De-flake sentry_client-test and sentry_authz_provider-test .. Patch Set 1: Verified+1 Unrelated flaky DefaultSourceTest.

[kudu-CR] De-flake sentry client-test and sentry authz provider-test

2018-11-06 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: De-flake sentry_client-test and sentry_authz_provider-test .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/11898 To

[kudu-CR] De-flake sentry client-test and sentry authz provider-test

2018-11-06 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11898 Change subject: De-flake sentry_client-test and sentry_authz_provider-test .. De-flake sentry_client-test and sentry_authz_provider-test

<    5   6   7   8   9   10   11   12   13   14   >