You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Hao Hao (Code Review)" <ge...@cloudera.org> on 2020/04/14 05:34:51 UTC
[kudu-CR](branch-1.12.x) KUDU-3078: parameterize ts sentry-itest to use Ranger
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15727
Change subject: KUDU-3078: parameterize ts_sentry-itest to use Ranger
......................................................................
KUDU-3078: parameterize ts_sentry-itest to use Ranger
Following the example set in dacb8203f05ca2d873d524501214bfe5144ac9bb,
this updates the test harness in ts_sentry-itest (now ts_authz-itest) to
be parameterized by authz provider.
Change-Id: I2232cfe0295cce83154cd01b5104c9d1b6ed955d
Reviewed-on: http://gerrit.cloudera.org:8080/15714
Reviewed-by: Hao Hao <ha...@cloudera.com>
Tested-by: Kudu Jenkins
(cherry picked from commit e6c742ec72d5da84554f1ab5f0e5c5603ab6307b)
---
M src/kudu/integration-tests/CMakeLists.txt
R src/kudu/integration-tests/ts_authz-itest.cc
2 files changed, 247 insertions(+), 92 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/27/15727/1
--
To view, visit http://gerrit.cloudera.org:8080/15727
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.12.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: I2232cfe0295cce83154cd01b5104c9d1b6ed955d
Gerrit-Change-Number: 15727
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
[kudu-CR](branch-1.12.x) KUDU-3078: parameterize ts sentry-itest to use Ranger
Posted by "Hao Hao (Code Review)" <ge...@cloudera.org>.
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15727 )
Change subject: KUDU-3078: parameterize ts_sentry-itest to use Ranger
......................................................................
KUDU-3078: parameterize ts_sentry-itest to use Ranger
Following the example set in dacb8203f05ca2d873d524501214bfe5144ac9bb,
this updates the test harness in ts_sentry-itest (now ts_authz-itest) to
be parameterized by authz provider.
Change-Id: I2232cfe0295cce83154cd01b5104c9d1b6ed955d
Reviewed-on: http://gerrit.cloudera.org:8080/15714
Reviewed-by: Hao Hao <ha...@cloudera.com>
Tested-by: Kudu Jenkins
(cherry picked from commit e6c742ec72d5da84554f1ab5f0e5c5603ab6307b)
Reviewed-on: http://gerrit.cloudera.org:8080/15727
Reviewed-by: Alexey Serbin <as...@cloudera.com>
Tested-by: Hao Hao <ha...@cloudera.com>
---
M src/kudu/integration-tests/CMakeLists.txt
R src/kudu/integration-tests/ts_authz-itest.cc
2 files changed, 247 insertions(+), 92 deletions(-)
Approvals:
Alexey Serbin: Looks good to me, approved
Hao Hao: Verified
--
To view, visit http://gerrit.cloudera.org:8080/15727
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.12.x
Gerrit-MessageType: merged
Gerrit-Change-Id: I2232cfe0295cce83154cd01b5104c9d1b6ed955d
Gerrit-Change-Number: 15727
Gerrit-PatchSet: 2
Gerrit-Owner: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
[kudu-CR](branch-1.12.x) KUDU-3078: parameterize ts sentry-itest to use Ranger
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15727 )
Change subject: KUDU-3078: parameterize ts_sentry-itest to use Ranger
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/15727
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.12.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I2232cfe0295cce83154cd01b5104c9d1b6ed955d
Gerrit-Change-Number: 15727
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 14 Apr 2020 05:49:00 +0000
Gerrit-HasComments: No
[kudu-CR](branch-1.12.x) KUDU-3078: parameterize ts sentry-itest to use Ranger
Posted by "Hao Hao (Code Review)" <ge...@cloudera.org>.
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/15727 )
Change subject: KUDU-3078: parameterize ts_sentry-itest to use Ranger
......................................................................
Patch Set 1: Verified+1
Unrelated flaky test.
--
To view, visit http://gerrit.cloudera.org:8080/15727
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.12.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I2232cfe0295cce83154cd01b5104c9d1b6ed955d
Gerrit-Change-Number: 15727
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 14 Apr 2020 17:51:58 +0000
Gerrit-HasComments: No
[kudu-CR](branch-1.12.x) KUDU-3078: parameterize ts sentry-itest to use Ranger
Posted by "Hao Hao (Code Review)" <ge...@cloudera.org>.
Hao Hao has removed a vote on this change.
Change subject: KUDU-3078: parameterize ts_sentry-itest to use Ranger
......................................................................
Removed Verified-1 by Kudu Jenkins (120)
--
To view, visit http://gerrit.cloudera.org:8080/15727
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.12.x
Gerrit-MessageType: deleteVote
Gerrit-Change-Id: I2232cfe0295cce83154cd01b5104c9d1b6ed955d
Gerrit-Change-Number: 15727
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)