Fang-Yu Rao has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/19373 )

Change subject: IMPALA-10399, IMPALA-11060, IMPALA-11788: Reset Ranger policy 
repository in an E2E test
......................................................................


Patch Set 3:

(2 comments)

Hi all, I have addressed Quanlong's comments in the previous iteration.

Let me know if there is any additional suggestion.

Thanks very much for the help!

http://gerrit.cloudera.org:8080/#/c/19373/2//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/19373/2//COMMIT_MSG@15
PS2, Line 15:  patch, th
> nit: download
Thanks Quanlong!

To emphasize this was the behavior prior to this patch, in the updated commit 
message, I will add "Before this patch" in front the this sentence.


http://gerrit.cloudera.org:8080/#/c/19373/2/testdata/bin/setup-ranger.sh
File testdata/bin/setup-ranger.sh:

http://gerrit.cloudera.org:8080/#/c/19373/2/testdata/bin/setup-ranger.sh@25
PS2, Line 25: function setup-ranger {
> Thanks for extracting this! It also fixes IMPALA-11060.
Ack



--
To view, visit http://gerrit.cloudera.org:8080/19373
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iff56ec03ceeb2912039241ea302f4bb8948d61f8
Gerrit-Change-Number: 19373
Gerrit-PatchSet: 3
Gerrit-Owner: Fang-Yu Rao <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Fang-Yu Rao <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Comment-Date: Wed, 21 Dec 2022 05:12:54 +0000
Gerrit-HasComments: Yes

Reply via email to