Abhishek Chennaka has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23205 )

Change subject: [Java] KUDU-3677: Fix optional dependencies
......................................................................


Patch Set 3:

Yes, that is correct but there could be a situation where 
def finalDeps = implDeps - shadowDeps
will not contain all the unshaded configuration dependencies which will result 
in having duplicates of these dependencies.
This could be when declaring dependencies under multiple scopes so having it 
this way makes it clear that all the compileUnshaded dependencies will be 
managed manually and need to be removed from the auto generated entities of the 
POM.


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7593f595b64082d86fb64cbe8f38ff9b2ccb88
Gerrit-Change-Number: 23205
Gerrit-PatchSet: 3
Gerrit-Owner: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Comment-Date: Thu, 24 Jul 2025 01:01:23 +0000
Gerrit-HasComments: No

Reply via email to