Maven
  1. Maven
  2. MNG-4062

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

        Hide
        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
        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
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: