Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12020 )

Change subject: IMPALA-7917 (Part 1): Initial work to decouple Sentry from 
Impala
......................................................................


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/12020/2/fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java
File fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java:

http://gerrit.cloudera.org:8080/#/c/12020/2/fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java@258
PS2, Line 258:       int initialHmsCnxnTimeoutSec, AuthorizationConfig 
authConfig, TUniqueId catalogServiceId,
line too long (95 > 90)


http://gerrit.cloudera.org:8080/#/c/12020/2/fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java
File fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java:

http://gerrit.cloudera.org:8080/#/c/12020/2/fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java@919
PS2, Line 919:     
TestWithIncorrectConfig(SentryAuthorizationConfig.createHadoopGroupAuthConfig("server1",
line too long (92 > 90)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If1fd1df0b38ddd7cfa41299e95f5827f8a9e9c1f
Gerrit-Change-Number: 12020
Gerrit-PatchSet: 2
Gerrit-Owner: Fredy Wijaya <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Sat, 01 Dec 2018 04:17:58 +0000
Gerrit-HasComments: Yes

Reply via email to