----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25211/ -----------------------------------------------------------
Review request for Falcon and shwethags. 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 ----- common/src/main/java/org/apache/falcon/entity/CatalogStorage.java 7ad0716 common/src/main/java/org/apache/falcon/entity/FileSystemStorage.java 4eb3d60 common/src/main/java/org/apache/falcon/entity/Storage.java f88e139 retention/src/main/java/org/apache/falcon/retention/FeedEvictor.java 4de7938 Diff: https://reviews.apache.org/r/25211/diff/ Testing ------- All FeedEvictor tests passed. Thanks, Ajay Yadava
