[kudu-CR] docs: add the required config for Sentry

2019-07-24 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13902 ) Change subject: docs: add the required config for Sentry .. docs: add the required config for Sentry This commit adds the required

[kudu-CR] docs: add the required config for Sentry

2019-07-24 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13902 ) Change subject: docs: add the required config for Sentry .. Patch Set 3: Verified+1 Unrelated flaky test -- To view, visit

[kudu-CR] docs: add the required config for Sentry

2019-07-24 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: docs: add the required config for Sentry .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13902 To unsubscribe, visit

[kudu-CR] docs: add the required config for Sentry

2019-07-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13902 ) Change subject: docs: add the required config for Sentry .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] docs: add the required config for Sentry

2019-07-23 Thread Hao Hao (Code Review)
Hello Alex Rodoni, Alexey Serbin, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13902 to look at the new patch set (#3). Change subject: docs: add the required config for Sentry

[kudu-CR] docs: add the required config for Sentry

2019-07-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13902 ) Change subject: docs: add the required config for Sentry .. Patch Set 3: (2 comments)

[kudu-CR] docs: add the required config for Sentry

2019-07-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13902 ) Change subject: docs: add the required config for Sentry .. Patch Set 2: Verified+1 (1 comment)

[kudu-CR] docs: add the required config for Sentry

2019-07-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13902 ) Change subject: docs: add the required config for Sentry .. Patch Set 2: -Verified Code-Review+1 -- To view, visit

[kudu-CR] docs: add the required config for Sentry

2019-07-23 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13902 ) Change subject: docs: add the required config for Sentry .. Patch Set 2: Code-Review+2 One nit question, otherwise LGTM -- To view,

[kudu-CR] docs: add the required config for Sentry

2019-07-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13902 ) Change subject: docs: add the required config for Sentry .. Patch Set 2: (2 comments)

[kudu-CR] docs: add the required config for Sentry

2019-07-23 Thread Hao Hao (Code Review)
Hello Alex Rodoni, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13902 to look at the new patch set (#2). Change subject: docs: add the required config for Sentry

[kudu-CR] docs: add the required config for Sentry

2019-07-23 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13902 ) Change subject: docs: add the required config for Sentry .. Patch Set 1: (2 comments)

[kudu-CR] docs: add the required config for Sentry

2019-07-23 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/13902 ) Change subject: docs: add the required config for Sentry .. Patch Set 1: (1 comment)

[kudu-CR] docs: add the required config for Sentry

2019-07-23 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13902 Change subject: docs: add the required config for Sentry .. docs: add the required config for Sentry Staged version here: