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

Minor improvements to Flume assembly config

        Details

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

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          mpercy Mike Percy added a comment -

          Improvements:

          • get rid of long file name warning
          • remove unneeded "inherited" tag
          • also stage files into a directory so when testing you don't have to untar the package to run flume at dev time
          Show
          mpercy Mike Percy added a comment - Improvements: get rid of long file name warning remove unneeded "inherited" tag also stage files into a directory so when testing you don't have to untar the package to run flume at dev time
          Hide
          Permalink
          mpercy Mike Percy added a comment -

          Tiny patch, skipping RB

          Show
          mpercy Mike Percy added a comment - Tiny patch, skipping RB
          Hide
          Permalink
          hshreedharan Hari Shreedharan added a comment -

          Is this the expected directory structure?
          flume-ng-dist/target/apache-flume-1.4.0-SNAPSHOT-bin/apache-flume-1.4.0-SNAPSHOT-bin
          flume-ng-dist/target/apache-flume-1.4.0-SNAPSHOT-src/apache-flume-1.4.0-SNAPSHOT-src

          Why is there an additional level?

          Show
          hshreedharan Hari Shreedharan added a comment - Is this the expected directory structure? flume-ng-dist/target/apache-flume-1.4.0-SNAPSHOT-bin/apache-flume-1.4.0-SNAPSHOT-bin flume-ng-dist/target/apache-flume-1.4.0-SNAPSHOT-src/apache-flume-1.4.0-SNAPSHOT-src Why is there an additional level?
          Hide
          Permalink
          mpercy Mike Percy added a comment -

          Hmm... I'm not sure. I don't really think it's a problem though, since it's mainly for convenience @ dev time. The artifacts only have a single level, as expected.

          Show
          mpercy Mike Percy added a comment - Hmm... I'm not sure. I don't really think it's a problem though, since it's mainly for convenience @ dev time. The artifacts only have a single level, as expected.
          Hide
          Permalink
          mpercy Mike Percy added a comment -

          I'm inclined to commit this as part of my 1.4 RM duties. Hari, if you have concerns please let me know ASAP. Thanks!

          Show
          mpercy Mike Percy added a comment - I'm inclined to commit this as part of my 1.4 RM duties. Hari, if you have concerns please let me know ASAP. Thanks!
          Hide
          Permalink
          hshreedharan Hari Shreedharan added a comment -

          +1. Please go ahead and commit this.

          Show
          hshreedharan Hari Shreedharan added a comment - +1. Please go ahead and commit this.
          Hide
          Permalink
          mpercy Mike Percy added a comment -

          I just pushed this to trunk and flume-1.4 branches. Thanks for taking a look Hari!

          Show
          mpercy Mike Percy added a comment - I just pushed this to trunk and flume-1.4 branches. Thanks for taking a look Hari!
          Hide
          Permalink
          hudson Hudson added a comment -

          Integrated in flume-trunk #445 (See https://builds.apache.org/job/flume-trunk/445/)
          FLUME-2005. Minor improvements to Flume assembly config. (Revision 171310fb1ab9316dd1d72126acbefd4ef4727043)

          Result = FAILURE
          mpercy : http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=171310fb1ab9316dd1d72126acbefd4ef4727043
          Files :

          • flume-ng-dist/src/main/assembly/bin.xml
          • flume-ng-dist/src/main/assembly/src.xml
          • flume-ng-dist/pom.xml
          Show
          hudson Hudson added a comment - Integrated in flume-trunk #445 (See https://builds.apache.org/job/flume-trunk/445/ ) FLUME-2005 . Minor improvements to Flume assembly config. (Revision 171310fb1ab9316dd1d72126acbefd4ef4727043) Result = FAILURE mpercy : http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=171310fb1ab9316dd1d72126acbefd4ef4727043 Files : flume-ng-dist/src/main/assembly/bin.xml flume-ng-dist/src/main/assembly/src.xml flume-ng-dist/pom.xml

            People

            • Assignee:
              mpercy Mike Percy
              Reporter:
              mpercy Mike Percy
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development