[ 
https://issues.apache.org/jira/browse/FALCON-145?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14148861#comment-14148861
 ] 

Suhas Vasu commented on FALCON-145:
-----------------------------------

[~ajayyadava] - Is the v2 patch attached here the latest ? I see more diffs 
uploaded in the review board.

> Feed eviction be implemented in appropriate Storage implementation
> ------------------------------------------------------------------
>
>                 Key: FALCON-145
>                 URL: https://issues.apache.org/jira/browse/FALCON-145
>             Project: Falcon
>          Issue Type: Improvement
>            Reporter: Venkatesh Seetharam
>            Assignee: Ajay Yadav
>         Attachments: FALCON-145-v2.patch, falcon-145.patch
>
>
> Since the feed storage is abstracted in Storage class either as 
> FileSystemStorage or CatalogStorage, moreover, behaviors for listing 
> partitions and drop partitions are listed there, why do we need to hardcode 
> the eviction behavior and instance deletion discovery for filesystem need to 
> happen in FeedEvictor ? Why can't be implemented in appropriate Storage 
> implementation. That way FeedEvictor would simpler and lot cleaner. 
> This can apply to table replication as well for import and export of 
> partitions.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to