Flume
  1. Flume
  2. FLUME-1980

Log4jAppender should optionally drop events if append fails

    Details

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

      Description

      We should allow a config to drop the event when append fails, rather than throwing an exception. Clients usually dont want the logging call to throw an exception and probably won't have code to handle this - especially when the code is some pre-packaged code and not their own.

      1. FLUME-1980.patch
        16 kB
        Hari Shreedharan

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        2d 10m 1 Hari Shreedharan 10/Apr/13 23:18
        Patch Available Patch Available Resolved Resolved
        29d 23h 45m 1 Mike Percy 10/May/13 23:03
        Hide
        Hudson added a comment -

        Integrated in flume-trunk #409 (See https://builds.apache.org/job/flume-trunk/409/)
        FLUME-1980. Log4jAppender should optionally drop events if append fails. (Revision 40040326763cdc655ea2707d8cb90a46791192c8)

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

        • flume-ng-clients/flume-ng-log4jappender/src/test/java/org/apache/flume/clients/log4jappender/TestLoadBalancingLog4jAppender.java
        • flume-ng-doc/sphinx/FlumeUserGuide.rst
        • flume-ng-clients/flume-ng-log4jappender/src/main/java/org/apache/flume/clients/log4jappender/Log4jAppender.java
        • flume-ng-clients/flume-ng-log4jappender/src/main/java/org/apache/flume/clients/log4jappender/LoadBalancingLog4jAppender.java
        • flume-ng-clients/flume-ng-log4jappender/src/test/java/org/apache/flume/clients/log4jappender/TestLog4jAppender.java
        Show
        Hudson added a comment - Integrated in flume-trunk #409 (See https://builds.apache.org/job/flume-trunk/409/ ) FLUME-1980 . Log4jAppender should optionally drop events if append fails. (Revision 40040326763cdc655ea2707d8cb90a46791192c8) Result = SUCCESS mpercy : http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=40040326763cdc655ea2707d8cb90a46791192c8 Files : flume-ng-clients/flume-ng-log4jappender/src/test/java/org/apache/flume/clients/log4jappender/TestLoadBalancingLog4jAppender.java flume-ng-doc/sphinx/FlumeUserGuide.rst flume-ng-clients/flume-ng-log4jappender/src/main/java/org/apache/flume/clients/log4jappender/Log4jAppender.java flume-ng-clients/flume-ng-log4jappender/src/main/java/org/apache/flume/clients/log4jappender/LoadBalancingLog4jAppender.java flume-ng-clients/flume-ng-log4jappender/src/test/java/org/apache/flume/clients/log4jappender/TestLog4jAppender.java
        Mike Percy made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Fix Version/s v1.4.0 [ 12323372 ]
        Resolution Fixed [ 1 ]
        Hide
        Mike Percy added a comment -

        Pushed to trunk and flume-1.4 branches.

        Thanks for the patch Hari!

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

        +1, looks good, let's get this in

        Show
        Mike Percy added a comment - +1, looks good, let's get this in
        Hide
        Roman Shaposhnik added a comment -

        +1 (non-bidning).

        Show
        Roman Shaposhnik added a comment - +1 (non-bidning).
        Hari Shreedharan made changes -
        Attachment FLUME-1980.patch [ 12578109 ]
        Hide
        Hari Shreedharan added a comment -

        Removed a file accidentally added file in the last patc

        Show
        Hari Shreedharan added a comment - Removed a file accidentally added file in the last patc
        Hari Shreedharan made changes -
        Attachment FLUME-1980.patch [ 12578107 ]
        Hari Shreedharan made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Hari Shreedharan made changes -
        Attachment FLUME-1980.patch [ 12578107 ]
        Hari Shreedharan made changes -
        Field Original Value New Value
        Summary Log4jAppender should optionally drop event if append fails Log4jAppender should optionally drop events if append fails
        Hari Shreedharan created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development