Maven
  1. Maven
  2. MNG-4179

[regression] Artifact download hangs upon transfer failure

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1.0
    • Fix Version/s: 2.2.0
    • Labels:
      None
    • Complexity:
      Intermediate
    • Number of attachments :
      0

      Description

      There's still a hole in the parallel artifact resolution we introduced in MNG-3379 for a scenario like this:

      • request to resolve multiple artifacts from the same groupId (i.e. all of these artifacts will be resolved by the same thread)
      • any but the last artifact of the group fails to resolve

      Upon error, the existing code will not try to resolve other artifacts in the same group, thereby never decrementing the countdown latch which guards the execution of the main thread.

        Issue Links

          Activity

          Hide
          Benjamin Bentmann added a comment -

          Fixed in r779768.

          Show
          Benjamin Bentmann added a comment - Fixed in r779768 .
          Hide
          Benjamin Bentmann added a comment -

          Merged to 2.1.1 in r779772.

          Show
          Benjamin Bentmann added a comment - Merged to 2.1.1 in r779772 .
          Hide
          Paul Benedict added a comment -

          How come this isn't fixed in the unreleased 2.2.0? At least according to JIRA...

          Show
          Paul Benedict added a comment - How come this isn't fixed in the unreleased 2.2.0? At least according to JIRA...
          Hide
          Benjamin Bentmann added a comment -

          The 2.2.0 branch was already in RC mode when I fixed the bug, so I left it to the release manager to merge it into 2.2.0.

          Show
          Benjamin Bentmann added a comment - The 2.2.0 branch was already in RC mode when I fixed the bug, so I left it to the release manager to merge it into 2.2.0.
          Hide
          John Casey added a comment -

          merged into 2.2.0-RC branch.

          Show
          John Casey added a comment - merged into 2.2.0-RC branch.

            People

            • Assignee:
              Benjamin Bentmann
              Reporter:
              Benjamin Bentmann
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: