Tim Armstrong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16225 )

Change subject: IMPALA-9980: Remove jersey* jars from maven exclusions
......................................................................


Patch Set 1:

(1 comment)

What was the test gap? Why didn't our ranger tests catch this?

How are we meant to prevent this regressing in future?

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

http://gerrit.cloudera.org:8080/#/c/16225/1/fe/pom.xml@189
PS1, Line 189:           <artifactId>jersey-bundle</artifactId>
This doesn't make a lot of sense to me - if the problem is with Ranger, why are 
we removing exclusions from non-Ranger dependencies and leaving the exclusion 
for ranger?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I796a9ff4c1beff776147266c2f2649c2d02a8dd8
Gerrit-Change-Number: 16225
Gerrit-PatchSet: 1
Gerrit-Owner: Anurag Mantripragada <anu...@cloudera.com>
Gerrit-Reviewer: Fang-Yu Rao <fangyu....@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Comment-Date: Tue, 21 Jul 2020 03:20:36 +0000
Gerrit-HasComments: Yes

Reply via email to