Maven 2 & 3

Evaluate possibility to cache parents during multi module build

Details

  • Type: Task Task
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 3.0-alpha-3
  • Fix Version/s: 3.0-alpha-3
  • Component/s: Performance
  • Labels:
    None
  • Complexity:
    Intermediate
  • Number of attachments :
    0

Description

From a breakpoint in DefaultMavenProjectBuilder.build(File, ProjectBuilderConfiguration) it seems that the parent of a multi-module project is rebuild for each child in the reactor. Given the likelihood of the modules in the reactor having the same parent, it could be a good opportunity for optimization to cache the parents (or projects in general).

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Benjamin Bentmann added a comment -

Of course, caching parents in general, i.e. parents of plugins or dependencies, should in general reduce project builder overhead.

Show
Benjamin Bentmann added a comment - Of course, caching parents in general, i.e. parents of plugins or dependencies, should in general reduce project builder overhead.
Brett Porter made changes -
Field Original Value New Value
Fix Version/s 3.0-alpha-5 [ 14952 ]
Jason van Zyl made changes -
Assignee Benjamin Bentmann [ bentmann ]
Fix Version/s 3.0-alpha-3 [ 14719 ]
Fix Version/s 3.0-alpha-5 [ 14952 ]
Hide
Permalink
Benjamin Bentmann added a comment -

The model builder is using a ModelCache to reuse parent POMs and imported POMs in a reactor and MavenProject.getParent() itself is not used by the core so no hotspot either.

Show
Benjamin Bentmann added a comment - The model builder is using a ModelCache to reuse parent POMs and imported POMs in a reactor and MavenProject.getParent() itself is not used by the core so no hotspot either.
Benjamin Bentmann made changes -
Status Open [ 1 ] Closed [ 6 ]
Resolution Fixed [ 1 ]

People

  • Assignee:
    Benjamin Bentmann
    Reporter:
    Benjamin Bentmann
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: