[kudu-CR] [sentry] add AuthzProvider

2018-11-05 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. [sentry] add AuthzProvider This commit adds a high-level abstraction which handles

[kudu-CR] [sentry] add AuthzProvider

2018-11-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 10: (1 comment)

[kudu-CR] [sentry] add AuthzProvider

2018-11-05 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 14: Code-Review+2 (1 comment)

[kudu-CR] [sentry] add AuthzProvider

2018-11-05 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 14: (1 comment)

[kudu-CR] [sentry] add AuthzProvider

2018-11-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 14: Code-Review+1 (1 comment)

[kudu-CR] [sentry] add AuthzProvider

2018-11-02 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 13: Code-Review+2 LGTM as long as Alexey's content -- To view, visit

[kudu-CR] [sentry] add AuthzProvider

2018-11-02 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11659 to look at the new patch set (#13). Change subject: [sentry] add AuthzProvider

[kudu-CR] [sentry] add AuthzProvider

2018-11-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 12: (1 comment)

[kudu-CR] [sentry] add AuthzProvider

2018-11-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 10: (1 comment)

[kudu-CR] [sentry] add AuthzProvider

2018-11-02 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11659 to look at the new patch set (#12). Change subject: [sentry] add AuthzProvider

[kudu-CR] [sentry] add AuthzProvider

2018-11-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 12: (3 comments)

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 11: (1 comment)

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 11: Code-Review+1 (2 comments) Sorry, a couple small nits and LGTM.

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 11: (1 comment)

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 11: (1 comment)

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11659 to look at the new patch set (#11). Change subject: [sentry] add AuthzProvider

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 10: (2 comments)

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 10: -Code-Review (2 comments)

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 10: Code-Review+1 (2 comments)

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 10: Code-Review+2 Looks good to me; maybe Andrew has some more feedback. --

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 10: (3 comments)

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11659 to look at the new patch set (#10). Change subject: [sentry] add AuthzProvider

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11659 to look at the new patch set (#9). Change subject: [sentry] add AuthzProvider

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 8: Code-Review+1 (3 comments)

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11659 to look at the new patch set (#8). Change subject: [sentry] add AuthzProvider

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 8: (6 comments)

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 7: (1 comment)

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 7: (1 comment)

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 7: (2 comments)

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 7: (9 comments)

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 7: (7 comments)

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 7: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/11659 To

[kudu-CR] [sentry] add AuthzProvider

2018-11-01 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 7: (7 comments) Mostly just a few nits at this point.

[kudu-CR] [sentry] add AuthzProvider

2018-10-31 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 7: (4 comments)

[kudu-CR] [sentry] add AuthzProvider

2018-10-31 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11659 to look at the new patch set (#7). Change subject: [sentry] add AuthzProvider

[kudu-CR] [sentry] add AuthzProvider

2018-10-31 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 6: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/11659 To

[kudu-CR] [sentry] add AuthzProvider

2018-10-31 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 6: (27 comments) http://gerrit.cloudera.org:8080/#/c/11659/5//COMMIT_MSG Commit

[kudu-CR] [sentry] add AuthzProvider

2018-10-31 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11659 to look at the new patch set (#6). Change subject: [sentry] add AuthzProvider

[kudu-CR] [sentry] add AuthzProvider

2018-10-30 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 5: (21 comments) http://gerrit.cloudera.org:8080/#/c/11659/5//COMMIT_MSG

[kudu-CR] [sentry] add AuthzProvider

2018-10-30 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 5: (6 comments) http://gerrit.cloudera.org:8080/#/c/11659/5//COMMIT_MSG Commit

[kudu-CR] [sentry] add AuthzProvider

2018-10-30 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 5: Verified+1 Unrelated flaky test

[kudu-CR] [sentry] add AuthzProvider

2018-10-30 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: [sentry] add AuthzProvider .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/11659 To unsubscribe, visit

[kudu-CR] [sentry] add AuthzProvider

2018-10-30 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 5: (32 comments) http://gerrit.cloudera.org:8080/#/c/11659/4//COMMIT_MSG Commit

[kudu-CR] [sentry] add AuthzProvider

2018-10-30 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11659 to look at the new patch set (#5). Change subject: [sentry] add AuthzProvider