Joe McDonnell created KUDU-3610:
-----------------------------------
Summary: KUDU-3551 introduced new unnecessary dependencies for
kudu-client.jar
Key: KUDU-3610
URL: https://issues.apache.org/jira/browse/KUDU-3610
Project: Kudu
Issue Type: Bug
Components: java
Affects Versions: 1.18.0
Reporter: Joe McDonnell
Attachments: kudu-client-a9d42cdb84.pom, kudu-client-e742f86f6d.pom
When working on fixing IMPALA-13309, I was upgrading Kudu from
[e742f86f6d|https://github.com/apache/kudu/commit/e742f86f6d] to
[a9d42cdb84|https://github.com/apache/kudu/commit/a9d42cdb8438858cf1bd18c203d1687ed5734346],
which is past the upgrade to Gradle 7.6.4 in
[e456bc7758|https://github.com/apache/kudu/commit/e456bc775805d4555d99ce12d04e3ca0b8950760]
(KUDU-3551). When using this new version, Impala's build fails with a message
like this:
{noformat}
15:24:49 [ERROR] Failed to execute goal on project impala-frontend: Could not
resolve dependencies for project
org.apache.impala:impala-frontend:jar:4.5.0-SNAPSHOT: Failed to collect
dependencies at org.apache.kudu:kudu-client:jar:a9d42cdb84 ->
org.apache.kudu:kudu-proto:jar:a9d42cdb84: Failed to read artifact descriptor
for org.apache.kudu:kudu-proto:jar:a9d42cdb84: Could not transfer artifact
org.apache.kudu:kudu-proto:pom:a9d42cdb84 from/to impala.cdp.repo (yadda yadda
yadda)
{noformat}
Impala builds and publishes kudu-client.jar, but we currently don't publish
kudu-proto.jar. This was fine on the old Kudu version, because kudu-client.jar
is a shaded jar and does not have a runtime dependency on kudu-proto.jar. After
the upgrade to Gradle 7.6.4, kudu-client.jar gains a runtime dependency on
kudu-proto.jar and a few other dependencies (guava, netty, etc). These extra
runtime dependencies should not be needed, because they are included in the
shaded jar.
I have attached the kudu-client.pom files from before and after for comparison:
Before: [^kudu-client-e742f86f6d.pom]
After: [^kudu-client-a9d42cdb84.pom]
To produce this file during development, run this from the java directory:
{noformat}
./gradlew -Dmaven.repo.local=/path/to/a/local/directory
:kudu-client:publishToMavenLocal{noformat}
Then, the new pom file will be under the
/path/to/a/local/directory/org/apache/kudu/kudu-client/1.18.0-SNAPSHOT/
directory.
It would be good for the dependencies to be as restricted as before. I think we
can get around some of these dependencies by using "compileOnly" rather than
"implementation".
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
