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

2019-03-25 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. [sentry] Integrate AuthzProvider into CatalogManager This

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

2019-03-25 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 16: Code-Review+2 -- To view, visit

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

2019-03-25 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/11797 To unsubscribe,

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

2019-03-25 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 16: Verified+1 Unrelated flaky

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

2019-03-24 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/11797 to look at the new patch set (#16). Change subject: [sentry] Integrate AuthzProvider into CatalogManager

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

2019-03-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 15: (1 comment)

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

2019-03-23 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 15: Code-Review+1 (1 comment)

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

2019-03-23 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/11797 To unsubscribe,

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

2019-03-23 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 15: Verified+1 unrelated flake (TSAN warnings in

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

2019-03-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 15: Code-Review+2 -- To view, visit

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

2019-03-22 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/11797 to look at the new patch set (#15). Change subject: [sentry] Integrate AuthzProvider into CatalogManager

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

2019-03-22 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 14: (2 comments)

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

2019-03-22 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 14: > (1 comment) > > Any appetite for addressing the

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

2019-03-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 14: (1 comment) Any appetite for addressing the

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

2019-03-22 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/11797 to look at the new patch set (#14). Change subject: [sentry] Integrate AuthzProvider into CatalogManager

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

2019-03-22 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 14: (12 comments)

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

2019-03-21 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 13: (12 comments) Just more nits and a couple test

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

2019-03-21 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 13: Code-Review+1 -- To view, visit

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

2019-03-21 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 13: Code-Review+1 -- To view, visit

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

2019-03-21 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/11797 to look at the new patch set (#13). Change subject: [sentry] Integrate AuthzProvider into CatalogManager

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

2019-03-21 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 12: (6 comments)

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

2019-03-19 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 12: (6 comments)

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

2019-03-19 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 11: (21 comments)

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

2019-03-19 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/11797 to look at the new patch set (#12). Change subject: [sentry] Integrate AuthzProvider into CatalogManager

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

2019-03-17 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 11: (8 comments) Mostly nits, and a couple

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

2019-03-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 11: (13 comments)

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

2019-03-15 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 10: (19 comments)

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

2019-03-15 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/11797 to look at the new patch set (#11). Change subject: [sentry] Integrate AuthzProvider into CatalogManager

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

2019-03-08 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 10: (9 comments)

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

2019-03-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 10: (9 comments)

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

2019-03-07 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 10: (1 comment)

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

2019-03-07 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 7: (62 comments)

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

2019-03-07 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/11797 to look at the new patch set (#10). Change subject: [sentry] Integrate AuthzProvider into CatalogManager

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

2019-02-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 7: (30 comments)

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

2019-02-09 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 7: (10 comments) Haven't gone deep into the tests

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

2019-02-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 7: (22 comments)

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

2019-02-07 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 7: (1 comment)

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

2019-02-07 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/11797 to look at the new patch set (#7). Change subject: [sentry] Integrate AuthzProvider into CatalogManager

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

2019-02-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 6: (17 comments) Just noticed a new patch set has

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

2019-02-07 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/11797 to look at the new patch set (#6). Change subject: [sentry] Integrate AuthzProvider into CatalogManager

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

2019-02-06 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 5: (1 comment)

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

2019-02-06 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/11797 to look at the new patch set (#5). Change subject: [sentry] Integrate AuthzProvider into CatalogManager

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

2019-02-06 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 4: (1 comment) Haven't reviewed anything, just

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

2019-02-04 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 4: (35 comments)

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

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

[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] [sentry] Integrate AuthzProvider into CatalogManager

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

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

2018-12-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 3: (33 comments)

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