Zoltan Chovan has posted comments on this change. ( http://gerrit.cloudera.org:8080/22768 )
Change subject: [java] KUDU-3657: fix publishing ...................................................................... Patch Set 17: > Patch Set 17: > > (1 comment) The main reason why many of the implementation scopes were changed is that when declaring them as 'api' these dependencies are not included in the generated pom.xml. This behaviour is a bit odd though, as from the documentation: "Dependencies appearing in the api configurations will be transitively exposed to consumers of the library, and as such will appear on the compile classpath of consumers. Dependencies found in the implementation configuration will, on the other hand, not be exposed to consumers, and therefore not leak into the consumers' compile classpath." Which seems to contradict the actual behavior of the configurations. So just to reiterate: when declaring for example "implementation libs.netty", then the netty dependency will be included in the pom as runtime dependency. These dependencies were not present at all before the gradle version upgrade (where the scopes compile and runtime are removed from gradle), so after discussing with Abhishek, we opted to switch to api since that resolved the pom issue. You can check the document I shared earlier about the pom differences, please note that the currently generated pom is much different than the "post-update" is in the document, that is basically after upgrade but before this patch. Maybe this is a discrepancy between gradle and maven scoping behaviour. What do you think? -- To view, visit http://gerrit.cloudera.org:8080/22768 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie045805a68bfcd16325e1e4dbb38653cefe3a4b0 Gerrit-Change-Number: 22768 Gerrit-PatchSet: 17 Gerrit-Owner: Zoltan Chovan <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Tue, 13 May 2025 07:34:34 +0000 Gerrit-HasComments: No
