[kudu-CR] [sentry] SentryAction

2018-10-13 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11656 ) Change subject: [sentry] SentryAction .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/11656/3/src/kudu/sentry/sentry_a

[kudu-CR] [sentry] SentryAction

2018-10-13 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/11656 to look at the new patch set (#3). Change subject: [sentry] SentryAction .. [se

[kudu-CR] [sentry] SentryAction

2018-10-13 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/11656 to look at the new patch set (#2). Change subject: [sentry] SentryAction .. [sentry] Sent

[kudu-CR] [sentry] SentryAction

2018-10-13 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11656 ) Change subject: [sentry] SentryAction .. Patch Set 1: (14 comments) Discussed with Dan offline, and we agreed that it is sufficient to only

[kudu-CR] [sentry] Fill out more sentry client API

2018-10-13 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/11657 to look at the new patch set (#2). Change subject: [sentry] Fill out more sentry client API ..

[kudu-CR] Add "service name" as part of ConnectionId

2018-10-13 Thread Michael Ho (Code Review)
Michael Ho has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11681 Change subject: Add "service_name" as part of ConnectionId .. Add "service_name" as part of ConnectionId The motivation for doing so is to

[kudu-CR] [tools] Add locate row tool

2018-10-13 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11666 ) Change subject: [tools] Add locate row tool .. Patch Set 10: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/11666/9/src/

[kudu-CR] [tools] Add a describe table tool

2018-10-13 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11665 ) Change subject: [tools] Add a describe table tool .. Patch Set 7: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/11665

[kudu-CR] [tools] Add a describe table tool

2018-10-13 Thread Will Berkeley (Code Review)
Will Berkeley has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/11665 ) Change subject: [tools] Add a describe table tool .. Removed reviewer Kudu Jenkins with the following votes: * Verified-1 by K

[kudu-CR] [tools] Add locate row tool

2018-10-13 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11666 ) Change subject: [tools] Add locate row tool .. Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c/11666/9/src/kudu/tools/to

[kudu-CR] [tools] Add locate row tool

2018-10-13 Thread Will Berkeley (Code Review)
Hello Mike Percy, Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11666 to look at the new patch set (#10). Change subject: [tools] Add locate row tool ...

[kudu-CR] [tools] Add a describe table tool

2018-10-13 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Mike Percy, Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11665 to look at the new patch set (#7). Change subject: [tools] Add a describe table tool .

[kudu-CR] [tools] Add a describe table tool

2018-10-13 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11665 ) Change subject: [tools] Add a describe table tool .. Patch Set 5: -Verified I compromised between making this compact, so it's easier

[kudu-CR] [tools] Add a describe table tool

2018-10-13 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Mike Percy, Alexey Serbin, Attila Bukor, Andrew Wong, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11665 to look at the new patch set (#6). Change subject: [tools] Add a describe table tool ...