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

(Updated Sept. 14, 2012, 2 p.m.)


Review request for Flume.


Changes
-------

Thank you Mike for the review.  

Summary of my changes
-Changed currentByteUsage to AtomicLong
-Also added synchronized(currentByteUsage) to put.


Description
-------

1. The user will be able to define a byteCapacity and a 
byteCapacityBufferPercentage.
2. Events byte size will be estimated from there body contents
3. On put bytes are added to current total
4. On commit any uncommitted takes are removed from the current byte total
5. On rollover any uncommitted puts are removed from the current byte total


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


Diffs (updated)
-----

  flume-ng-core/src/main/java/org/apache/flume/channel/MemoryChannel.java 
c72e97c 
  flume-ng-core/src/test/java/org/apache/flume/channel/TestMemoryChannel.java 
e070864 

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


Testing
-------


Thanks,

Ted Malaska

Reply via email to