-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Java] java-jars fails with "We have duplicated artifacts attached" #41490
Comments
Would it be the recent maven build changes, my best guess would be the last build PR, but I can't be sure. Though I didn't notice it before that. |
Checking my recent crossbow usage 8 days ago: passes 6 days ago: fails |
I know this doesn't help, but at least it should have happened sometime after 8 days ago. |
Do you think you could bisect it? Or at least figure out which commits are possible causes in that time range? |
That's due to the source maven plugin update coming with new Apache POM. On some profile, the source and jar maven plugins are executed by the parent Apache POM and by the project pom. So we have a dual execution, so dual artifacts. It's what I saw on Apache projects. I gonna take a look if it's the same here in Arrow. |
@lidavidm I feel like this is the PR which could have broken the CIs. Though java-jars pass in this PR itself, a change made in another PR followed by this merge could have triggered it. It is very hard to predict this since in PR #41309 the java-jars failure is clouded by the C++ filesystem issue which was fixed recently. But that PR has some changes in plugins. But I cannot exactly pin-point what change is causing it. cc @jbonofre WDYT? |
Yes, this is the PR I am also suspicious about. I am going to wait for your response. Thanks for looking into this @jbonofre |
I think, we hit the same issue as in my PR (#39215). We had similar issue there (due to Apache POM version bump as well). Let me double check if it's the same and eventually leverage the reproducible build PR to fix that 😄 |
This will become a release blocker at some point but I suppose 17.0.0 is still a bit off |
@lidavidm agree, so let me tackle that ! At your service 😄 |
It's much appreciated :) |
@lidavidm my pleasure, always happy to help 😄 |
The problem is in arrow/ci/scripts/java_full_build.sh Line 50 in c8f89d0
It explicitely calls IMHO, the
If agree, I can create a PR to change this pipeline. |
Ok, we can test in CI and confirm. Thanks for digging into this! |
Describe the bug, including details regarding any error messages, version, and platform.
https://github.com/ursacomputing/crossbow/actions/runs/8903205653/job/24451424677#step:6:16247
Component(s)
Java
The text was updated successfully, but these errors were encountered: