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

(Updated Aug. 24, 2012, 7:33 p.m.)


Review request for Flume, Arvind Prabhakar and Brock Noland.


Changes
-------

Made changes based on Brock's feedback to move the set of inflight file ids to 
the InflightEventWrapper class.


Description
-------

Keep track of file ids of non-committed transactions also. Remove them only 
when the transaction gets committed.


This addresses bug FLUME-1428.
    https://issues.apache.org/jira/browse/FLUME-1428


Diffs (updated)
-----

  
flume-ng-channels/flume-file-channel/src/main/java/org/apache/flume/channel/file/FlumeEventQueue.java
 766c59a 
  
flume-ng-channels/flume-file-channel/src/test/java/org/apache/flume/channel/file/TestFileChannel.java
 2093839 

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


Testing
-------

Added a unit test for this case.


Thanks,

Hari Shreedharan

Reply via email to