Uploaded image for project: 'Mahout'
  1. Mahout
  2. MAHOUT-622

Mahout dependencies are unified under dependency management in parent pom

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 0.4
    • Fix Version/s: 0.5
    • Component/s: None
    • Labels:

      Description

      As far as I understand, Maven encourages "best practice" of unified view of dependency versions specified under <dependencyManagement> usually in a parent pom, instead of under <dependencies>.

      In Mahout, this practice is only partially followed. Some dependencies have concrete versions under <dependencies> tag in submodule poms. Proposed change is to raid those and move version declarations into parent pom.

      This (as far as i understand) achieves 2 things:

      • Mahout assembly would include same versions for all modules thus ensuring runtime module dependencies are the same as compile time;
      • Somebody who uses Mahout as a dependency, could import Mahout dependencies using <scope>import</scope> spec thus inheriting Mahout's versions for shared dependencies.

      For most part the change would be nominal although in certain cases we'd need to sort out through cross-module conflicts (if any). Commons-math was one, not sure if there are more. If there are none, the changes would be rather mechanistic.

        Attachments

        1. MAHOUT-622-2.patch
          12 kB
          Dmitriy Lyubimov
        2. MAHOUT-622-utils.patch
          3 kB
          Dmitriy Lyubimov
        3. MAHOUT-622.patch
          11 kB
          Dmitriy Lyubimov
        4. MAHOUT-622.patch
          11 kB
          Dmitriy Lyubimov

          Issue Links

            Activity

              People

              • Assignee:
                dlyubimov Dmitriy Lyubimov
                Reporter:
                dlyubimov Dmitriy Lyubimov
              • Votes:
                1 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: