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

[MNG-8520] Add cache for model resolution during project building #2047

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

gnodet
Copy link
Contributor

@gnodet gnodet commented Jan 14, 2025

No description provided.

@cstamas
Copy link
Member

cstamas commented Jan 15, 2025

Tested locally, commented on https://issues.apache.org/jira/browse/MNG-8520 (tl;dr: it restores previous behaviour, excess of events are gone)

@cstamas
Copy link
Member

cstamas commented Jan 15, 2025

@alextu Can you confirm?

@cstamas cstamas added this to the 4.0.0-rc-3 milestone Jan 15, 2025
@gnodet gnodet marked this pull request as ready for review January 15, 2025 13:57
@alextu
Copy link

alextu commented Jan 15, 2025

@alextu Can you confirm?

It looks like it's better, using the reproducer I sent. I'll be able to confirm once it's merged as our test suite runs with our own nightly built from Maven master.

@cstamas
Copy link
Member

cstamas commented Jan 15, 2025

And from where you get the nightly build? You are building it yourselves?

@alextu
Copy link

alextu commented Jan 15, 2025

And from where you get the nightly build? You are building it yourselves?

yes on our internal CI. If really needed, I can check tomorrow how to tweak that to build a custom branch instead of master and use that in our tests.

@alextu
Copy link

alextu commented Jan 16, 2025

I've built this branch:

mvn --version
Apache Maven 4.0.0-rc-3-SNAPSHOT (129afd91931d6cb8688ab6f03d82c43e6ceb0815)

and run our tests, they're still failing for the same reason(s) 🤔: additional requests that are not traced back to plugins.

@cstamas
Copy link
Member

cstamas commented Jan 16, 2025

"additional requests that are not traced back to plugins" -- isnt this the https://issues.apache.org/jira/browse/MNG-8519 ? Or "additional" as "count of them is still more than was in mvn3"?

@alextu
Copy link

alextu commented Jan 16, 2025

"additional requests that are not traced back to plugins" -- isnt this the https://issues.apache.org/jira/browse/MNG-8519 ? Or "additional" as "count of them is still more than was in mvn3"?

The count of them is more, for example, in this test, we see 153 requests with this Maven 4 version and 117 with Maven 3.9.9. We count a request when we receive a pair of RepositoryEvent objects with type ARTIFACT_DOWNLOADING (or METADATA_DOWNLOADING) and ARTIFACT_DOWNLOADED (or METADATA_DOWNLOADED).
As part of this test, we also see that some (all?) of those additional requests cannot be traced back to a plugin or a project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants