Flume
  1. Flume
  2. FLUME-1955

fileSuffix does not work with compressed streams

    Details

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

      Description

      Compressed streams are excluded from fileSuffix support.

        Issue Links

          Activity

          Hide
          Hudson added a comment -

          Integrated in flume-trunk #380 (See https://builds.apache.org/job/flume-trunk/380/)
          FLUME-1955. fileSuffix does not work with compressed streams. (Revision 2f6fea50965e5a8ae059d179b0b25be738696cef)

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

          • flume-ng-sinks/flume-hdfs-sink/src/main/java/org/apache/flume/sink/hdfs/HDFSEventSink.java
          • flume-ng-sinks/flume-hdfs-sink/src/test/java/org/apache/flume/sink/hdfs/TestBucketWriter.java
          • flume-ng-sinks/flume-hdfs-sink/src/main/java/org/apache/flume/sink/hdfs/BucketWriter.java
          Show
          Hudson added a comment - Integrated in flume-trunk #380 (See https://builds.apache.org/job/flume-trunk/380/ ) FLUME-1955 . fileSuffix does not work with compressed streams. (Revision 2f6fea50965e5a8ae059d179b0b25be738696cef) Result = FAILURE hshreedharan : http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=2f6fea50965e5a8ae059d179b0b25be738696cef Files : flume-ng-sinks/flume-hdfs-sink/src/main/java/org/apache/flume/sink/hdfs/HDFSEventSink.java flume-ng-sinks/flume-hdfs-sink/src/test/java/org/apache/flume/sink/hdfs/TestBucketWriter.java flume-ng-sinks/flume-hdfs-sink/src/main/java/org/apache/flume/sink/hdfs/BucketWriter.java
          Hide
          Hari Shreedharan added a comment -

          Patch committed. Thanks Mike!

          Show
          Hari Shreedharan added a comment - Patch committed. Thanks Mike!
          Hide
          Hari Shreedharan added a comment -

          +1. Looks good. Will run tests and commit.

          Show
          Hari Shreedharan added a comment - +1. Looks good. Will run tests and commit.
          Hide
          Mike Percy added a comment -

          Patch and unit test to address this issue.

          Show
          Mike Percy added a comment - Patch and unit test to address this issue.

            People

            • Assignee:
              Mike Percy
              Reporter:
              Mike Percy
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development