Vihang Karajgaonkar has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/17874 )

Change subject: Bump up the GBN to 17296101
......................................................................


Patch Set 1:

(1 comment)

LGTM. I left a minor comment below. I can +2 this once it is addressed.

http://gerrit.cloudera.org:8080/#/c/17874/1/fe/pom.xml
File fe/pom.xml:

http://gerrit.cloudera.org:8080/#/c/17874/1/fe/pom.xml@122
PS1, Line 122:         <exclusion>
             :           <groupId>org.apache.ranger</groupId>
             :           <artifactId>ranger-plugins-audit</artifactId>
             :         </exclusion>
can you please add a few lines comments here similar to other exclusions. 
Specifically, what was the problem when you included this.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I87a497882e80dbfc87077bdbc2f05216182003d6
Gerrit-Change-Number: 17874
Gerrit-PatchSet: 1
Gerrit-Owner: Yu-Wen Lai <yu-wen....@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Vihang Karajgaonkar <vih...@cloudera.com>
Gerrit-Reviewer: Yu-Wen Lai <yu-wen....@cloudera.com>
Gerrit-Comment-Date: Tue, 28 Sep 2021 22:04:31 +0000
Gerrit-HasComments: Yes

Reply via email to