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

Aaron Fabbri commented on HADOOP-14488:
---------------------------------------

Thanks for the excellent work here [~mackrorysd].   I applied the patch, added 
some comments, removed a stray {{System.out.println()}} that was left in.  I 
also added a Precondition check to a helper function and renamed a variable or 
two (child -> ancestor).  Went through the code and it makes sense to me.  Ran 
{{ITestS3GuardListConsistency}} successfully in us-west-2. I'm cool with those 
test cases staying there, since they do use the inconsistency injection.

Please review my updated patch and I will commit it if you are +1.

As an aside, I think we need to address the inconsistency tests being slow, due 
to code like this:

{code}
+    Thread.sleep(2 * DEFAULT_DELAY_KEY_MSEC);
{code}

I have an idea for it (exposing a clear() or flush() function) and will address 
in a separate JIRA.


> s3guard listStatus fails after renaming file into directory
> -----------------------------------------------------------
>
>                 Key: HADOOP-14488
>                 URL: https://issues.apache.org/jira/browse/HADOOP-14488
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/s3
>            Reporter: Steve Loughran
>            Assignee: Sean Mackrory
>            Priority: Blocker
>         Attachments: HADOOP-14488-HADOOP-13345-001.patch, 
> HADOOP-14488-HADOOP-13345-002.patch, HADOOP-14488-HADOOP-13345-003.patch, 
> HADOOP-14488-HADOOP-13345-004.patch, output.txt
>
>
> Running scala integration test with inconsistent s3 client & local DDB enabled
> {code}
> fs.rename("work/task-00/part-00", work)
> fs.listStatus(work)
> {code}
> The list status work fails with a message about the childStatus not being a 
> child of the parent. 
> Hypothesis: rename isn't updating the child path entry



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
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