Skip to content

Advised and Enforced Management#1613

Closed
cstamas wants to merge 19 commits intoapache:masterfrom
cstamas:platform
Closed

Advised and Enforced Management#1613
cstamas wants to merge 19 commits intoapache:masterfrom
cstamas:platform

Conversation

@cstamas
Copy link
Copy Markdown
Member

@cstamas cstamas commented Oct 2, 2025

Originally, dependency management marked a "subject" as "managed", it was 0/1 true/false only. This PR now adds ability to mark management as "advised" (may be subjected to further management) or "enforced" (equivalent of previously "managed").

Changes:

  • DependencyManager applies management as "enforced", if they come from root (ie. project POM), otherwise as "advice" (this latter applies to transitive managers only, as for example classic manager have no data from "deeper levels").
  • ConflictManager backs out for enforced scope/optionality.
  • This makes the "isInheritedDerived" hack in dependency manager not needed, as now management rule tells the modality of it (is enforced or just advised)

@cstamas cstamas self-assigned this Oct 2, 2025
@cstamas cstamas changed the title Platform support Advised and Enforced Management Oct 2, 2025
@cstamas cstamas mentioned this pull request Oct 2, 2025
@cstamas cstamas requested a review from gnodet October 7, 2025 09:51
@cstamas cstamas added the enhancement New feature or request label Oct 7, 2025
@cstamas cstamas added this to the 2.0.14 milestone Oct 7, 2025
@cstamas cstamas marked this pull request as ready for review October 7, 2025 10:11
@gnodet
Copy link
Copy Markdown
Contributor

gnodet commented Oct 30, 2025

Any idea about memory/perfs implications of the switch to Map?

@cstamas
Copy link
Copy Markdown
Member Author

cstamas commented Oct 30, 2025

Did not perf measure it yet, but obviously will have some impact: none of code change affect long living objects, except the DefaultDependencyNode, where instead of byte we now have a Map. Will need to measure it.

All the rest of code changes is either very short living (DependencyManagement, PremanagedDependency) objects, or no effect like in case of DependencyManager instances...

@cstamas cstamas removed this from the 2.0.14 milestone Oct 30, 2025
@cstamas
Copy link
Copy Markdown
Member Author

cstamas commented Mar 27, 2026

Superseded by #1815

@cstamas cstamas closed this Mar 27, 2026
@cstamas cstamas deleted the platform branch March 27, 2026 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants