-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49453/
-----------------------------------------------------------

(Updated July 13, 2016, 12:29 p.m.)


Review request for Flume, Balázs Donát Bessenyei and Attila Simon.


Changes
-------

Fixes based on Mike's review


Bugs: FLUME-2725
    https://issues.apache.org/jira/browse/FLUME-2725


Repository: flume-git


Description
-------

Patch for FLUME-2725 - HDFS Sink does not use configured timezone for rounding


Diffs (updated)
-----

  flume-ng-core/src/main/java/org/apache/flume/formatter/output/BucketPath.java 
b2fe3f0 
  
flume-ng-core/src/main/java/org/apache/flume/tools/TimestampRoundDownUtil.java 
daa9606 
  
flume-ng-core/src/test/java/org/apache/flume/formatter/output/TestBucketPath.java
 b1b828a 
  
flume-ng-core/src/test/java/org/apache/flume/tools/TestTimestampRoundDownUtil.java
 1ac11ab 

Diff: https://reviews.apache.org/r/49453/diff/


Testing
-------

`org.apache.flume.formatter.output.TestBucketPath` and 
`org.apache.flume.tools.TestTimestampRoundDownUtil` were extended with new 
methods testing with `TimeZone`. Existing and new tests pass.


Thanks,

Denes Arvay

Reply via email to