Flume
  1. Flume
  2. FLUME-902

Remove thrift references in NG build

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: NG alpha 1
    • Fix Version/s: v1.0.0
    • Component/s: Build
    • Labels:
      None

      Description

      Remove use / references to thrift stuff from the build. We don't currently use it and it just adds to build cruft.

        Activity

        Hide
        jiraposter@reviews.apache.org added a comment -

        -----------------------------------------------------------
        This is an automatically generated e-mail. To reply, visit:
        https://reviews.apache.org/r/3311/
        -----------------------------------------------------------

        Review request for Flume.

        Summary
        -------

        Remove thrift plugin and dep references. We don't use them.

        This addresses bug FLUME-902.
        https://issues.apache.org/jira/browse/FLUME-902

        Diffs


        c/flume-ng-core/pom.xml 2726b06
        c/pom.xml 3a177f0

        Diff: https://reviews.apache.org/r/3311/diff

        Testing
        -------

        Ran the build and all tests.

        Thanks,

        Eric

        Show
        jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3311/ ----------------------------------------------------------- Review request for Flume. Summary ------- Remove thrift plugin and dep references. We don't use them. This addresses bug FLUME-902 . https://issues.apache.org/jira/browse/FLUME-902 Diffs c/flume-ng-core/pom.xml 2726b06 c/pom.xml 3a177f0 Diff: https://reviews.apache.org/r/3311/diff Testing ------- Ran the build and all tests. Thanks, Eric
        Hide
        jiraposter@reviews.apache.org added a comment -

        -----------------------------------------------------------
        This is an automatically generated e-mail. To reply, visit:
        https://reviews.apache.org/r/3311/#review4124
        -----------------------------------------------------------

        Ship it!

        +1

        • Arvind

        On 2011-12-24 05:14:02, Eric Sammer wrote:

        -----------------------------------------------------------

        This is an automatically generated e-mail. To reply, visit:

        https://reviews.apache.org/r/3311/

        -----------------------------------------------------------

        (Updated 2011-12-24 05:14:02)

        Review request for Flume.

        Summary

        -------

        Remove thrift plugin and dep references. We don't use them.

        This addresses bug FLUME-902.

        https://issues.apache.org/jira/browse/FLUME-902

        Diffs

        -----

        c/flume-ng-core/pom.xml 2726b06

        c/pom.xml 3a177f0

        Diff: https://reviews.apache.org/r/3311/diff

        Testing

        -------

        Ran the build and all tests.

        Thanks,

        Eric

        Show
        jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3311/#review4124 ----------------------------------------------------------- Ship it! +1 Arvind On 2011-12-24 05:14:02, Eric Sammer wrote: ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3311/ ----------------------------------------------------------- (Updated 2011-12-24 05:14:02) Review request for Flume. Summary ------- Remove thrift plugin and dep references. We don't use them. This addresses bug FLUME-902 . https://issues.apache.org/jira/browse/FLUME-902 Diffs ----- c/flume-ng-core/pom.xml 2726b06 c/pom.xml 3a177f0 Diff: https://reviews.apache.org/r/3311/diff Testing ------- Ran the build and all tests. Thanks, Eric
        Hide
        E. Sammer added a comment -

        Committed to flume-728.

        Show
        E. Sammer added a comment - Committed to flume-728.
        Hide
        Hudson added a comment -

        Integrated in flume-728 #84 (See https://builds.apache.org/job/flume-728/84/)
        FLUME-902: Remove thrift references in NG build

        esammer : http://svn.apache.org/viewvc/?view=rev&rev=1225064
        Files :

        • /incubator/flume/branches/flume-728/flume-ng-core/pom.xml
        • /incubator/flume/branches/flume-728/pom.xml
        Show
        Hudson added a comment - Integrated in flume-728 #84 (See https://builds.apache.org/job/flume-728/84/ ) FLUME-902 : Remove thrift references in NG build esammer : http://svn.apache.org/viewvc/?view=rev&rev=1225064 Files : /incubator/flume/branches/flume-728/flume-ng-core/pom.xml /incubator/flume/branches/flume-728/pom.xml

          People

          • Assignee:
            E. Sammer
            Reporter:
            E. Sammer
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development