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

Aaron Fabbri commented on HADOOP-13631:
---------------------------------------

Hi [~eddyxu] thanks for the review.

{quote}
Can we move this logic into LocalMetadataStore and keep MetadataStore#move(src, 
dst) as it is?
{quote}

I would have prefered that simple approach, but recall the discussion on 
HADOOP-13448 (starting 
[here|https://issues.apache.org/jira/browse/HADOOP-13448?focusedCommentId=15510928&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15510928]).

Give that a re-read and let me know what you think.

> S3Guard: implement move() for LocalMetadataStore, add unit tests
> ----------------------------------------------------------------
>
>                 Key: HADOOP-13631
>                 URL: https://issues.apache.org/jira/browse/HADOOP-13631
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/s3
>            Reporter: Aaron Fabbri
>            Assignee: Aaron Fabbri
>         Attachments: HADOOP-13631-HADOOP-13345.001.patch
>
>
> Building on HADOOP-13573 and HADOOP-13452, implement move() in 
> LocalMetadataStore and associated MetadataStore unit tests.
> (Making this a separate JIRA to break up work into decent-sized and 
> reviewable chunks.)



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

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to