Flume
  1. Flume
  2. FLUME-2198

Avro Source should disable itself if ipFilterRules contains invalid rules

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: v1.5.0
    • Fix Version/s: v1.5.0
    • Component/s: None
    • Labels:
      None
    1. FLUME-2198.patch
      12 kB
      Hari Shreedharan
    2. FLUME-2198.patch
      12 kB
      Hari Shreedharan
    3. FLUME-2198-2.patch
      14 kB
      Hari Shreedharan
    4. FLUME-2198-3.patch
      17 kB
      Hari Shreedharan

      Issue Links

        Activity

        Hide
        Hari Shreedharan added a comment -

        Patch that parses the rules on configuration and throws if the rules are not valid.

        Show
        Hari Shreedharan added a comment - Patch that parses the rules on configuration and throws if the rules are not valid.
        Hide
        Hari Shreedharan added a comment -

        Re-enabling some tests which were accidentally disabled.

        Show
        Hari Shreedharan added a comment - Re-enabling some tests which were accidentally disabled.
        Hide
        Hari Shreedharan added a comment -

        Also made sure the source is stopped each time to prevent resource leaks that were killing the tests every now and then.

        Show
        Hari Shreedharan added a comment - Also made sure the source is stopped each time to prevent resource leaks that were killing the tests every now and then.
        Hide
        Hari Shreedharan added a comment -

        Update based on Mike's feedback on rb

        Show
        Hari Shreedharan added a comment - Update based on Mike's feedback on rb
        Hide
        Hari Shreedharan added a comment -

        Updating to use loopback IP

        Show
        Hari Shreedharan added a comment - Updating to use loopback IP
        Hide
        Mike Percy added a comment -

        +1 looks good. I am about to commit this.

        Show
        Mike Percy added a comment - +1 looks good. I am about to commit this.
        Hide
        Mike Percy added a comment -

        Pushed to trunk and flume-1.5 branches. Thanks for the patch Hari!

        Show
        Mike Percy added a comment - Pushed to trunk and flume-1.5 branches. Thanks for the patch Hari!
        Hide
        Hudson added a comment -

        SUCCESS: Integrated in flume-trunk #503 (See https://builds.apache.org/job/flume-trunk/503/)
        FLUME-2198. Avro Source should disable itself if ipFilterRules contains invalid rules (mpercy: http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=49933493f53633fec5846edf88aca4dd0dfdd52a)

        • flume-ng-core/src/main/java/org/apache/flume/source/AvroSource.java
        • flume-ng-core/src/test/java/org/apache/flume/source/TestAvroSource.java
        Show
        Hudson added a comment - SUCCESS: Integrated in flume-trunk #503 (See https://builds.apache.org/job/flume-trunk/503/ ) FLUME-2198 . Avro Source should disable itself if ipFilterRules contains invalid rules (mpercy: http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=49933493f53633fec5846edf88aca4dd0dfdd52a ) flume-ng-core/src/main/java/org/apache/flume/source/AvroSource.java flume-ng-core/src/test/java/org/apache/flume/source/TestAvroSource.java

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development