----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25211/ -----------------------------------------------------------
(Updated Sept. 13, 2014, 4:43 a.m.) Review request for Falcon and shwethags. Changes ------- Rebased the patch. Enumerated the imports. Repository: falcon-git Description ------- Earlier FeedEviction class contained logic for both FileSystemStorage and TableStorage. Corresponding code was being called using an if else. To make the code cleaner and more manageable I moved the code to the appropriate storage class and delegated feed eviction to the appropriate Storage implementation. Needed to add evict method to Storage Interface and make some minor changes here and there. Diffs (updated) ----- common/src/main/java/org/apache/falcon/entity/CatalogStorage.java 7ad0716 common/src/main/java/org/apache/falcon/entity/FileSystemStorage.java 58506ad common/src/main/java/org/apache/falcon/entity/Storage.java f88e139 common/src/main/java/org/apache/falcon/retention/EvictionHelper.java 5d6481c retention/src/main/java/org/apache/falcon/retention/FeedEvictor.java 114071f Diff: https://reviews.apache.org/r/25211/diff/ Testing ------- All FeedEvictor tests passed. Thanks, Ajay Yadava
