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

      Hide
      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
      mpercy Mike Percy added a comment -

      Tiny patch, skipping RB

      Show
      mpercy Mike Percy added a comment - Tiny patch, skipping RB
      Hide
      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
      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
      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
      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
      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
      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