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

ThriftSource throws NPE in stop() if start() failed because socket open failed or if thrift server instance creation threw.

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.4.0
    • Component/s: None
    • Labels:
      None
    1. FLUME-2025.patch
      1 kB
      Hari Shreedharan

      Activity

      Hide
      hshreedharan Hari Shreedharan added a comment -

      Enclosing servingExecutor cleanup in a not null check. Let me know if this has to go on rb too.

      Show
      hshreedharan Hari Shreedharan added a comment - Enclosing servingExecutor cleanup in a not null check. Let me know if this has to go on rb too.
      Hide
      mpercy Mike Percy added a comment -

      +1 looks good

      Show
      mpercy Mike Percy added a comment - +1 looks good
      Hide
      mpercy Mike Percy added a comment -

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

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

      Integrated in flume-trunk #426 (See https://builds.apache.org/job/flume-trunk/426/)
      FLUME-2025. ThriftSource throws NPE in stop() if start() failed because socket open failed or if thrift server instance creation threw. (Revision 296fc9f92b51bbe9c674cede560ab7be8a03e161)

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

      • flume-ng-core/src/main/java/org/apache/flume/source/ThriftSource.java
      Show
      hudson Hudson added a comment - Integrated in flume-trunk #426 (See https://builds.apache.org/job/flume-trunk/426/ ) FLUME-2025 . ThriftSource throws NPE in stop() if start() failed because socket open failed or if thrift server instance creation threw. (Revision 296fc9f92b51bbe9c674cede560ab7be8a03e161) Result = SUCCESS mpercy : http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=296fc9f92b51bbe9c674cede560ab7be8a03e161 Files : flume-ng-core/src/main/java/org/apache/flume/source/ThriftSource.java

        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