Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/23205 )
Change subject: [Java] KUDU-3677: Fix optional dependencies ...................................................................... Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/23205/1/java/gradle/publishing.gradle File java/gradle/publishing.gradle: http://gerrit.cloudera.org:8080/#/c/23205/1/java/gradle/publishing.gradle@101 PS1, Line 101: nit: stray space symbols http://gerrit.cloudera.org:8080/#/c/23205/1/java/gradle/publishing.gradle@103 PS1, Line 103: nit: remove the trailing space http://gerrit.cloudera.org:8080/#/c/23205/1/java/gradle/publishing.gradle@102 PS1, Line 102: def filteredImplDeps = implDeps - shadowDeps - unshadedDeps : def finalDeps = filteredImplDeps + unshadedDeps Wouldn't it be the same if having just def finalDeps = implDeps - shadowDeps ? http://gerrit.cloudera.org:8080/#/c/23205/1/java/gradle/publishing.gradle@127 PS1, Line 127: if (!dep.excludeRules.isEmpty()) { : def exclusionsNode = depNode.appendNode("exclusions") : dep.excludeRules.each { ex -> : def exclusion = exclusionsNode.appendNode("exclusion") : exclusion.appendNode("groupId", ex.group ?: "*") : exclusion.appendNode("artifactId", ex.module ?: "*") Did you verify that we don't have anything else but exclusions that could require special handling as this for all other components we have in the unshaded deps? -- 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: 1 Gerrit-Owner: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Tue, 22 Jul 2025 05:43:36 +0000 Gerrit-HasComments: Yes
