Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11531 )
Change subject: IMPALA-7503: SHOW GRANT USER not showing all privileges. ...................................................................... Patch Set 8: (6 comments) http://gerrit.cloudera.org:8080/#/c/11531/8/tests/authorization/test_owner_privileges.py File tests/authorization/test_owner_privileges.py: http://gerrit.cloudera.org:8080/#/c/11531/8/tests/authorization/test_owner_privileges.py@118 PS8, Line 118: flake8: W291 trailing whitespace http://gerrit.cloudera.org:8080/#/c/11531/8/tests/authorization/test_owner_privileges.py@118 PS8, Line 118: impalad_args="--server_name=server1 --sentry_config={0} " line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/11531/8/tests/authorization/test_owner_privileges.py@234 PS8, Line 234: flake8: W291 trailing whitespace http://gerrit.cloudera.org:8080/#/c/11531/8/tests/authorization/test_owner_privileges.py@234 PS8, Line 234: impalad_args="--server_name=server1 --sentry_config={0} " line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/11531/8/tests/authorization/test_owner_privileges.py@307 PS8, Line 307: flake8: W291 trailing whitespace http://gerrit.cloudera.org:8080/#/c/11531/8/tests/authorization/test_owner_privileges.py@307 PS8, Line 307: impalad_args="--server_name=server1 --sentry_config={0} " line has trailing whitespace -- To view, visit http://gerrit.cloudera.org:8080/11531 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie9f6c88f5569e1c414ceb8a86e7b013eaa3ecde1 Gerrit-Change-Number: 11531 Gerrit-PatchSet: 8 Gerrit-Owner: Adam Holley <[email protected]> Gerrit-Reviewer: Adam Holley <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Fri, 28 Sep 2018 06:58:37 +0000 Gerrit-HasComments: Yes
