[kudu-CR] KUDU-2514 Part 2: Supports setting history max age sec for the specified table

2019-06-10 Thread Yao Xu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13573 to look at the new patch set (#4). Change subject: KUDU-2514 Part 2: Supports setting history_max_age_sec for the specified table

[kudu-CR] KUDU-2514 Part 2: Supports setting history max age sec for the specified table

2019-06-10 Thread Yao Xu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13573 to look at the new patch set (#3). Change subject: KUDU-2514 Part 2: Supports setting history_max_age_sec for the specified table

[kudu-CR] [rebalance] Add '--ignored tservers' flag to rebalancer

2019-06-10 Thread Yifan Zhang (Code Review)
Yifan Zhang has posted comments on this change. ( http://gerrit.cloudera.org:8080/13539 ) Change subject: [rebalance] Add '--ignored_tservers' flag to rebalancer .. Patch Set 5: (3 comments)

[kudu-CR] [rebalance] Add '--ignored tservers' flag to rebalancer

2019-06-10 Thread Yifan Zhang (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13539 to look at the new patch set (#5). Change subject: [rebalance] Add '--ignored_tservers' flag to rebalancer

[kudu-CR] KUDU-2797: the master aggregates tablet metrics

2019-06-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13426 ) Change subject: KUDU-2797: the master aggregates tablet metrics .. Patch Set 5: (10 comments)

[kudu-CR] [itbll] make error/failure easier to detect

2019-06-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13575 ) Change subject: [itbll] make error/failure easier to detect .. Patch Set 3: (1 comment)

[kudu-CR] hms: clarify message to pass in --hive metastore uris

2019-06-10 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13578 ) Change subject: hms: clarify message to pass in --hive_metastore_uris .. hms: clarify message to pass in --hive_metastore_uris

[kudu-CR] hms: clarify message to pass in --hive metastore uris

2019-06-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13578 ) Change subject: hms: clarify message to pass in --hive_metastore_uris .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] hms: clarify message to pass in --hive metastore uris

2019-06-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13578 ) Change subject: hms: clarify message to pass in --hive_metastore_uris .. Patch Set 2: (1 comment)

[kudu-CR] hms: clarify message to pass in --hive metastore uris

2019-06-10 Thread Andrew Wong (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13578 to look at the new patch set (#2). Change subject: hms: clarify message to pass in --hive_metastore_uris

[kudu-CR] hms: clarify message to pass in --hive metastore uris

2019-06-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13578 ) Change subject: hms: clarify message to pass in --hive_metastore_uris .. Patch Set 1: (1 comment)

[kudu-CR] hms: clarify message to pass in --hive metastore uris

2019-06-10 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13578 ) Change subject: hms: clarify message to pass in --hive_metastore_uris .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] sentry: avoid authorizing every table in ListTables

2019-06-10 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13549 ) Change subject: sentry: avoid authorizing every table in ListTables .. sentry: avoid authorizing every table in ListTables

[kudu-CR] hms: clarify message to pass in --hive metastore uris

2019-06-10 Thread Andrew Wong (Code Review)
Hello Alexey Serbin, Hao Hao, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/13578 to review the following change. Change subject: hms: clarify message to pass in --hive_metastore_uris ..

[kudu-CR] thirdparty: fix build curl with unusual krb5-config location

2019-06-10 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13577 ) Change subject: thirdparty: fix build_curl with unusual krb5-config location .. thirdparty: fix build_curl with unusual

[kudu-CR] KUDU-2834: Build Kudu Kubernetes cluster using Helm chart

2019-06-10 Thread Sandish Kumar HN (Code Review)
Sandish Kumar HN has posted comments on this change. ( http://gerrit.cloudera.org:8080/13568 ) Change subject: KUDU-2834: Build Kudu Kubernetes cluster using Helm chart .. Patch Set 7: Created health-test.yaml, which will

[kudu-CR] sentry: avoid authorizing every table in ListTables

2019-06-10 Thread Andrew Wong (Code Review)
Andrew Wong has removed a vote on this change. Change subject: sentry: avoid authorizing every table in ListTables .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13549 To

[kudu-CR] sentry: avoid authorizing every table in ListTables

2019-06-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13549 ) Change subject: sentry: avoid authorizing every table in ListTables .. Patch Set 8: Verified+1 DEBUG: unrelated

[kudu-CR] KUDU-2834: Build Kudu Kubernetes cluster using Helm chart

2019-06-10 Thread Sandish Kumar HN (Code Review)
Hello Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13568 to look at the new patch set (#7). Change subject: KUDU-2834: Build Kudu Kubernetes cluster using Helm chart

[kudu-CR] thirdparty: fix build curl with unusual krb5-config location

2019-06-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13577 ) Change subject: thirdparty: fix build_curl with unusual krb5-config location .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes

2019-06-10 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13552 ) Change subject: sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes .. sentry: allow caching of

[kudu-CR] sentry: avoid authorizing every table in ListTables

2019-06-10 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13549 ) Change subject: sentry: avoid authorizing every table in ListTables .. Patch Set 8: Code-Review+2 -- To view, visit

[kudu-CR] thirdparty: fix build curl with unusual krb5-config location

2019-06-10 Thread Adar Dembo (Code Review)
Hello Alexey Serbin, Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/13577 to review the following change. Change subject: thirdparty: fix build_curl with unusual krb5-config location

[kudu-CR] authz: refactor authorization for ListTables

2019-06-10 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13566 ) Change subject: authz: refactor authorization for ListTables .. authz: refactor authorization for ListTables Authorization for

[kudu-CR] authz: refactor authorization for ListTables

2019-06-10 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13566 ) Change subject: authz: refactor authorization for ListTables .. Patch Set 6: Code-Review+2 -- To view, visit

[kudu-CR] [itbll] make error/failure easier to detect

2019-06-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13575 ) Change subject: [itbll] make error/failure easier to detect .. Patch Set 3: (1 comment)

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

2019-06-10 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13540 ) Change subject: [hms] Adjust storage handler package .. [hms] Adjust storage handler package This patch changes the new

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

2019-06-10 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: [hms] Adjust storage handler package .. Removed Verified-1 by Grant Henke -- To view, visit http://gerrit.cloudera.org:8080/13540 To unsubscribe, visit

[kudu-CR] sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes

2019-06-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13552 ) Change subject: sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes .. Patch Set 7: Code-Review+2 -- To

[kudu-CR] authz: refactor authorization for ListTables

2019-06-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13566 ) Change subject: authz: refactor authorization for ListTables .. Patch Set 6: Code-Review+1 -- To view, visit

[kudu-CR] KUDU-2834: Build Kudu Kubernetes cluster using Helm chart

2019-06-10 Thread Sandish Kumar HN (Code Review)
Hello Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13568 to look at the new patch set (#6). Change subject: KUDU-2834: Build Kudu Kubernetes cluster using Helm chart

[kudu-CR] sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes

2019-06-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13552 ) Change subject: sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes .. Patch Set 7: Code-Review+1 -- To view,

[kudu-CR] sentry: avoid authorizing every table in ListTables

2019-06-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13549 ) Change subject: sentry: avoid authorizing every table in ListTables .. Patch Set 8: Code-Review+1 -- To view, visit

[kudu-CR] authz: refactor authorization for ListTables

2019-06-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13566 ) Change subject: authz: refactor authorization for ListTables .. Patch Set 6: (2 comments)

[kudu-CR] sentry: avoid authorizing every table in ListTables

2019-06-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13549 ) Change subject: sentry: avoid authorizing every table in ListTables .. Patch Set 8: (1 comment)

[kudu-CR] authz: refactor authorization for ListTables

2019-06-10 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13566 to look at the new patch set (#6). Change subject: authz: refactor authorization for ListTables

[kudu-CR] [itbll] make error/failure easier to detect

2019-06-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13575 ) Change subject: [itbll] make error/failure easier to detect .. [itbll] make error/failure easier to detect I found it

[kudu-CR] [itbll] make error/failure easier to detect

2019-06-10 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13575 ) Change subject: [itbll] make error/failure easier to detect .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes

2019-06-10 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13552 ) Change subject: sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes .. Patch Set 6: Code-Review+2 -- To view,

[kudu-CR] KUDU-2834: Build Kudu Kubernetes cluster using Helm chart

2019-06-10 Thread Sandish Kumar HN (Code Review)
Sandish Kumar HN has posted comments on this change. ( http://gerrit.cloudera.org:8080/13568 ) Change subject: KUDU-2834: Build Kudu Kubernetes cluster using Helm chart .. Patch Set 4: (3 comments) I'm thinking of adding

[kudu-CR] KUDU-2834: Build Kudu Kubernetes cluster using Helm chart

2019-06-10 Thread Sandish Kumar HN (Code Review)
Hello Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13568 to look at the new patch set (#5). Change subject: KUDU-2834: Build Kudu Kubernetes cluster using Helm chart

[kudu-CR] KUDU-2834: Build Kudu Kubernetes cluster using Helm chart

2019-06-10 Thread Sandish Kumar HN (Code Review)
Sandish Kumar HN has abandoned this change. ( http://gerrit.cloudera.org:8080/13576 ) Change subject: KUDU-2834: Build Kudu Kubernetes cluster using Helm chart .. Abandoned duplicate -- To view, visit

[kudu-CR] KUDU-2834: Build Kudu Kubernetes cluster using Helm chart

2019-06-10 Thread Sandish Kumar HN (Code Review)
Sandish Kumar HN has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13576 Change subject: KUDU-2834: Build Kudu Kubernetes cluster using Helm chart .. KUDU-2834: Build Kudu Kubernetes cluster using Helm

[kudu-CR] [rebalance] Add '--ignored tservers' flag to rebalancer

2019-06-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13539 ) Change subject: [rebalance] Add '--ignored_tservers' flag to rebalancer .. Patch Set 4: (1 comment)

[kudu-CR] authz: refactor authorization for ListTables

2019-06-10 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13566 ) Change subject: authz: refactor authorization for ListTables .. Patch Set 5: Code-Review+1 (1 comment)

[kudu-CR] [itbll] make error/failure easier to detect

2019-06-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13575 ) Change subject: [itbll] make error/failure easier to detect .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [itbll] make error/failure easier to detect

2019-06-10 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/13575 to look at the new patch set (#2). Change subject: [itbll] make error/failure easier to detect

[kudu-CR] [rebalance] Add '--ignored tservers' flag to rebalancer

2019-06-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13539 ) Change subject: [rebalance] Add '--ignored_tservers' flag to rebalancer .. Patch Set 4: (3 comments)

[kudu-CR] [itbll] make error/failure easier to detect

2019-06-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13575 ) Change subject: [itbll] make error/failure easier to detect .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [itbll] make error/failure easier to detect

2019-06-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13575 Change subject: [itbll] make error/failure easier to detect .. [itbll] make error/failure easier to detect I found it extremely hard

[kudu-CR] KUDU-2514 Part 2: Supports setting history max age sec for the specified table

2019-06-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13573 ) Change subject: KUDU-2514 Part 2: Supports setting history_max_age_sec for the specified table .. Patch Set 2: (2 comments)

[kudu-CR] sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes

2019-06-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13552 ) Change subject: sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes .. Patch Set 6: Code-Review+1 -- To view,

[kudu-CR] sentry: avoid authorizing every table in ListTables

2019-06-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13549 ) Change subject: sentry: avoid authorizing every table in ListTables .. Patch Set 7: (1 comment)

[kudu-CR] authz: refactor authorization for ListTables

2019-06-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13566 ) Change subject: authz: refactor authorization for ListTables .. Patch Set 5: (1 comment)

[kudu-CR] KUDU-2834: Build Kudu Kubernetes cluster using Helm chart

2019-06-10 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13568 ) Change subject: KUDU-2834: Build Kudu Kubernetes cluster using Helm chart .. Patch Set 4: (10 comments) Thanks for the contribution! I

[kudu-CR](gh-pages) [site] Add new committer (laiyingchun)

2019-06-10 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13572 ) Change subject: [site] Add new committer (laiyingchun) .. [site] Add new committer (laiyingchun) Change-Id:

[kudu-CR](gh-pages) [site] Add new committer (laiyingchun)

2019-06-10 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13572 ) Change subject: [site] Add new committer (laiyingchun) .. Patch Set 1: Verified+1 -- To view, visit

[kudu-CR](gh-pages) [site] Add new committer (laiyingchun)

2019-06-10 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13572 ) Change subject: [site] Add new committer (laiyingchun) .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2514 Part 2: Supports setting history max age sec for the specified table

2019-06-10 Thread Yao Xu (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13573 to look at the new patch set (#2). Change subject: KUDU-2514 Part 2: Supports setting history_max_age_sec for the specified table

[kudu-CR] KUDU-2514 Part 2: Supports setting history max age sec for the specified table

2019-06-10 Thread Yao Xu (Code Review)
Yao Xu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13573 Change subject: KUDU-2514 Part 2: Supports setting history_max_age_sec for the specified table .. KUDU-2514 Part 2: Supports setting