Details

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

      Description

      It looks to be the channels variable which can be null:

      at org.apache.flume.conf.source.SourceConfiguration.configure(SourceConfiguration.java:62)
      ... 17 more

      1. FLUME-1847-0.patch
        2 kB
        Brock Noland
      2. FLUME-1847-1.patch
        2 kB
        Brock Noland

        Activity

        Hide
        hudson Hudson added a comment -

        Integrated in flume-trunk #357 (See https://builds.apache.org/job/flume-trunk/357/)
        FLUME-1847. NPE in SourceConfiguration (Revision f32e8cd4c529b1b334d8adf0282647eac3472294)

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

        • flume-ng-configuration/src/main/java/org/apache/flume/conf/source/SourceConfiguration.java
        • flume-ng-configuration/src/test/java/org/apache/flume/conf/source/TestSourceConfiguration.java
        Show
        hudson Hudson added a comment - Integrated in flume-trunk #357 (See https://builds.apache.org/job/flume-trunk/357/ ) FLUME-1847 . NPE in SourceConfiguration (Revision f32e8cd4c529b1b334d8adf0282647eac3472294) Result = SUCCESS harishreedharan : http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=f32e8cd4c529b1b334d8adf0282647eac3472294 Files : flume-ng-configuration/src/main/java/org/apache/flume/conf/source/SourceConfiguration.java flume-ng-configuration/src/test/java/org/apache/flume/conf/source/TestSourceConfiguration.java
        Hide
        hshreedharan Hari Shreedharan added a comment -

        Patch resolved, rev: f32e8cd4c529b1b334d8adf0282647eac3472294. Thanks Brock!

        Show
        hshreedharan Hari Shreedharan added a comment - Patch resolved, rev: f32e8cd4c529b1b334d8adf0282647eac3472294. Thanks Brock!
        Hide
        brocknoland Brock Noland added a comment -

        Fixed, thanks Hari!

        Show
        brocknoland Brock Noland added a comment - Fixed, thanks Hari!
        Hide
        hshreedharan Hari Shreedharan added a comment -

        Actually the unit test would fail right? Looks like you need to do

        @Test(expected=ConfigurationException.class)
        

        Can you fix that? The code looks good.

        Show
        hshreedharan Hari Shreedharan added a comment - Actually the unit test would fail right? Looks like you need to do @Test(expected=ConfigurationException.class) Can you fix that? The code looks good.
        Hide
        hshreedharan Hari Shreedharan added a comment -

        +1. Will run test and commit.

        Show
        hshreedharan Hari Shreedharan added a comment - +1. Will run test and commit.
        Hide
        brocknoland Brock Noland added a comment -

        Trivial patch, no RB item.

        Show
        brocknoland Brock Noland added a comment - Trivial patch, no RB item.

          People

          • Assignee:
            brocknoland Brock Noland
            Reporter:
            brocknoland Brock Noland
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development