Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/13525 )
Change subject: IMPALA-8551: Bump CDP_BUILD_NUMBER to 1153860 ...................................................................... Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/13525/2/tests/authorization/test_ranger.py File tests/authorization/test_ranger.py: http://gerrit.cloudera.org:8080/#/c/13525/2/tests/authorization/test_ranger.py@566 PS2, Line 566: ctional.alltypes from user {0}" : .format(getuser())]: : result = self.execute_query_expect_failure(invalid_impala_cli > it's weird that you need two different clients, but you're passing the user This is a known issue. I filed a bug here: https://issues.apache.org/jira/browse/IMPALA-8640 http://gerrit.cloudera.org:8080/#/c/13525/2/tests/authorization/test_ranger.py@578 PS2, Line 578: > is there any way we can get rid of the HTTP error part of the error message We can but we have to parse the error message and that can be somewhat error prone. Ranger throws a generic exception, so we pretty much just rely on the error message returned by them. -- To view, visit http://gerrit.cloudera.org:8080/13525 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8995f5dc88b211cd3af415713802cfeac44fe576 Gerrit-Change-Number: 13525 Gerrit-PatchSet: 3 Gerrit-Owner: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Fri, 07 Jun 2019 16:30:15 +0000 Gerrit-HasComments: Yes
