Skip to content
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

2.7.6 is timing out #324

Closed
rcfja opened this issue Apr 3, 2023 · 4 comments
Closed

2.7.6 is timing out #324

rcfja opened this issue Apr 3, 2023 · 4 comments
Assignees
Labels

Comments

@rcfja
Copy link

rcfja commented Apr 3, 2023

$ mvn -B org.cyclonedx:cyclonedx-maven-plugin:2.7.6:makeAggregateBom

...

[INFO]
[INFO] --------------------< com.XXX >---------------------
[INFO] Building XXX 5.9.5-SNAPSHOT
[INFO]   from pom.xml
[INFO] --------------------------------[ war ]---------------------------------
[INFO]
[INFO] --- cyclonedx:2.7.6:makeAggregateBom (default-cli) @ XXX  ---
[INFO] CycloneDX: Resolving Dependencies

And its just stuck at this point and never finishes and eventually my CI/CD times out on it after 30 mins.

If I downgrade to 2.7.5 it all works fine again. The generate only take 1 or 2 minutes to run.

Running on openjdk 1.8.0_362

@rcfja rcfja changed the title 2.7.6 is freezing on me 2.7.6 is timing out Apr 3, 2023
@hboutemy hboutemy added the bug label Apr 4, 2023
@hboutemy
Copy link
Contributor

hboutemy commented Apr 4, 2023

thanks for your report: we'll need your help to try to find the root cause (@knrc , I suppose we'll need your help)
Are you able to share with us a reproducer project?

@knrc
Copy link
Contributor

knrc commented Apr 4, 2023

@hboutemy I came across something similar when developing the PR for #306. I managed to change the way component resolution worked to reduce a ~30 min aggregate down to around 2 mins, but it's in #306 .
I'll take a look at extracting those parts later this week. I'm traveling today (~25 hours) so it may take a day or two.

Can you assign this to me?

@knrc
Copy link
Contributor

knrc commented Apr 4, 2023

BTW I have a project where this is evident, so I don't think I need a reproducer

@knrc
Copy link
Contributor

knrc commented Apr 11, 2023

I've submitted PR #333 which should address the performance issue. Note this PR doesn't address the dependency graph problems which we were trying to fix in #306.

hboutemy added a commit that referenced this issue Apr 13, 2023
Fix performance issue for aggregates, fixes #324
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants