Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11314 )
Change subject: IMPALA-7074: Update OWNER privilege on CREATE, DROP, and SET OWNER ...................................................................... Patch Set 28: (3 comments) http://gerrit.cloudera.org:8080/#/c/11314/28/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java File fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java: http://gerrit.cloudera.org:8080/#/c/11314/28/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java@1492 PS28, Line 1492: updateDatabasePrivileges(table.getDb().getName(), table.getName(), params.server_name, line too long (92 > 90) http://gerrit.cloudera.org:8080/#/c/11314/28/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java@3183 PS28, Line 3183: removedGrantOptPrivileges = catalog_.getSentryProxy().revokeRolePrivileges(requestingUser, line too long (96 > 90) http://gerrit.cloudera.org:8080/#/c/11314/28/tests/authorization/test_grant_revoke.py File tests/authorization/test_grant_revoke.py: http://gerrit.cloudera.org:8080/#/c/11314/28/tests/authorization/test_grant_revoke.py@32 PS28, Line 32: ' flake8: E501 line too long (91 > 90 characters) -- To view, visit http://gerrit.cloudera.org:8080/11314 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1e09332e007ed5aa6a0840683c879a8295c3d2b0 Gerrit-Change-Number: 11314 Gerrit-PatchSet: 28 Gerrit-Owner: Adam Holley <ahol...@cloudera.com> Gerrit-Reviewer: Adam Holley <ahol...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Vuk Ercegovac <vercego...@cloudera.com> Gerrit-Comment-Date: Mon, 10 Sep 2018 10:17:43 +0000 Gerrit-HasComments: Yes