Uploaded image for project: 'Flume'
  1. Flume
  2. FLUME-2567

Remove unneeded repository declarations in pom.xml

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.7.0
    • Component/s: None
    • Labels:
      None

      Description

      pom.xml contains multiple repository declarations that do not seem to be needed (jboss, maven central, cloudera). Actually, it seems to me that none of these is needed.

      Can we remove all repository declarations and let it use just the default (Maven Central)?

      1. FLUME-2567-0.patch
        4 kB
        Santiago M. Mola

        Issue Links

          Activity

          Hide
          fwiffo Joey Echeverria added a comment -

          Hey Santiago M. Mola, did you test this patch with a clean environment? I plan on testing the patch later today, but I wanted to see what you'd already done.

          Show
          fwiffo Joey Echeverria added a comment - Hey Santiago M. Mola , did you test this patch with a clean environment? I plan on testing the patch later today, but I wanted to see what you'd already done.
          Hide
          smolav Santiago M. Mola added a comment - - edited

          Joey Echeverria Yes. I've tested it on a clean environment (https://travis-ci.org/Stratio/flume/builds/42887413). Local Maven repository should be empty before every build.

          Show
          smolav Santiago M. Mola added a comment - - edited Joey Echeverria Yes. I've tested it on a clean environment ( https://travis-ci.org/Stratio/flume/builds/42887413 ). Local Maven repository should be empty before every build.
          Hide
          liorze Lior Zeno added a comment -

          Why hasn't this patch been committed?

          Show
          liorze Lior Zeno added a comment - Why hasn't this patch been committed?
          Hide
          mpercy Mike Percy added a comment -

          The squeaky wheel gets the grease.

          +1, LGTM. I am about to commit this.

          Show
          mpercy Mike Percy added a comment - The squeaky wheel gets the grease. +1, LGTM. I am about to commit this.
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 4eacba1930c3abfddf4fb50c5a272798708df811 in flume's branch refs/heads/trunk from Santiago M. Mola
          [ https://git-wip-us.apache.org/repos/asf?p=flume.git;h=4eacba1 ]

          FLUME-2567. Remove external Maven repositories

          (Santiago M. Mola via Mike Percy)

          Show
          jira-bot ASF subversion and git services added a comment - Commit 4eacba1930c3abfddf4fb50c5a272798708df811 in flume's branch refs/heads/trunk from Santiago M. Mola [ https://git-wip-us.apache.org/repos/asf?p=flume.git;h=4eacba1 ] FLUME-2567 . Remove external Maven repositories (Santiago M. Mola via Mike Percy)
          Hide
          mpercy Mike Percy added a comment -

          Pushed to trunk. Thanks for the patch, Santiago!

          Show
          mpercy Mike Percy added a comment - Pushed to trunk. Thanks for the patch, Santiago!
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Flume-trunk-hbase-1 #168 (See https://builds.apache.org/job/Flume-trunk-hbase-1/168/)
          FLUME-2567. Remove external Maven repositories (mpercy: http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=4eacba1930c3abfddf4fb50c5a272798708df811)

          • flume-ng-sinks/flume-ng-morphline-solr-sink/pom.xml
          • pom.xml
          • flume-ng-sinks/flume-dataset-sink/pom.xml
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Flume-trunk-hbase-1 #168 (See https://builds.apache.org/job/Flume-trunk-hbase-1/168/ ) FLUME-2567 . Remove external Maven repositories (mpercy: http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=4eacba1930c3abfddf4fb50c5a272798708df811 ) flume-ng-sinks/flume-ng-morphline-solr-sink/pom.xml pom.xml flume-ng-sinks/flume-dataset-sink/pom.xml

            People

            • Assignee:
              smolav Santiago M. Mola
              Reporter:
              smolav Santiago M. Mola
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development