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

Review request for Flume.


Summary
-------

Modified configure to check for an exisitng deque, and copy across data if the 
size has changed. If the new capacity is smaller than the number of items 
remaining in the old one, data is still lost, with a logger warning.


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


Diffs
-----

  
/branches/flume-728/flume-ng-core/src/main/java/org/apache/flume/channel/MemoryChannel.java
 1228002 
  
/branches/flume-728/flume-ng-core/src/test/java/org/apache/flume/channel/TestMemoryChannel.java
 1228002 

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


Testing
-------

Created new unit test to check assumptions made along with capacity limits.
New unit test passes
All existing unit tests are fine(except for ExecSource which fails because of 
development environment)


Thanks,

Juhani

Reply via email to