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

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        29m 35s 1 E. Sammer 24/Dec/11 05:14
        Patch Available Patch Available Resolved Resolved
        3d 19h 36m 1 E. Sammer 28/Dec/11 00:50
        Resolved Resolved Closed Closed
        4s 1 E. Sammer 28/Dec/11 00:50
        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
        E. Sammer made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        E. Sammer made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        E. Sammer added a comment -

        Committed to flume-728.

        Show
        E. Sammer added a comment - Committed to flume-728.
        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
        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
        E. Sammer made changes -
        Field Original Value New Value
        Status Open [ 1 ] Patch Available [ 10002 ]
        E. Sammer created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development