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

Add deprecation back to the legacy thrift code

    Details

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

      Description

      FLUME-1896 brought in new generated code, which removed the @Deprecated annotations.

      1. FLUME-1911.patch
        2 kB
        Hari Shreedharan

        Activity

        Hide
        hudson Hudson added a comment -

        Integrated in flume-trunk #377 (See https://builds.apache.org/job/flume-trunk/377/)
        FLUME-1911. Add deprecation back to the legacy thrift code. (Revision fc0b9b8cd27ebe2ff764d10f441cc332a6629f52)

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

        • flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/ThriftFlumeEventServer.java
        • flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/ThriftFlumeEvent.java
        Show
        hudson Hudson added a comment - Integrated in flume-trunk #377 (See https://builds.apache.org/job/flume-trunk/377/ ) FLUME-1911 . Add deprecation back to the legacy thrift code. (Revision fc0b9b8cd27ebe2ff764d10f441cc332a6629f52) Result = FAILURE mpercy : http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=fc0b9b8cd27ebe2ff764d10f441cc332a6629f52 Files : flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/ThriftFlumeEventServer.java flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/ThriftFlumeEvent.java
        Hide
        mpercy Mike Percy added a comment -

        Pushed to trunk & flume-1.4 branches. Thanks Hari!

        Show
        mpercy Mike Percy added a comment - Pushed to trunk & flume-1.4 branches. Thanks Hari!
        Hide
        mpercy Mike Percy added a comment -

        +1

        Show
        mpercy Mike Percy added a comment - +1
        Hide
        hshreedharan Hari Shreedharan added a comment -

        Trivial patch, skipping rb

        Show
        hshreedharan Hari Shreedharan added a comment - Trivial patch, skipping rb

          People

          • Assignee:
            hshreedharan Hari Shreedharan
            Reporter:
            hshreedharan Hari Shreedharan
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development