Hello Zoltan Chovan, Alexey Serbin,

I'd like you to do a code review. Please visit

    http://gerrit.cloudera.org:8080/23176

to review the following change.


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

[Java] KUDU-3677: Fix optional dependencies

This update removes compileUnshaded dependencies from the generated POM and 
re-adds them with
compile scope, including any defined exclusions without the optional 
configuration attribute.

We now use project.configurations.findByName("compileUnshaded") instead of 
direct access to
avoid Could not get unknown property 'compileUnshaded' errors when the 
configuration is not
defined.

Change-Id: I5b7593f595b64082d86fb64cbe8f38ff9b2ccb88
Reviewed-on: http://gerrit.cloudera.org:8080/23205
Reviewed-by: Zoltan Chovan <[email protected]>
Tested-by: Abhishek Chennaka <[email protected]>
Reviewed-by: Alexey Serbin <[email protected]>
(cherry picked from commit 35fce9ba24b2e376c65fd7a2c50a65ec0a33e02f)
---
M java/gradle/publishing.gradle
1 file changed, 60 insertions(+), 10 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/76/23176/2
--
To view, visit http://gerrit.cloudera.org:8080/23176
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.18.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5b7593f595b64082d86fb64cbe8f38ff9b2ccb88
Gerrit-Change-Number: 23176
Gerrit-PatchSet: 2
Gerrit-Owner: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>

Reply via email to