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

Sean Mackrory commented on HADOOP-13760:
----------------------------------------

Looks like infra issues. For what it's worth, this is the output of my Yetus 
run locally:

{code}| Vote |      Subsystem |  Runtime   | Comment
============================================================================
|  +1  |       @author  |   0m  0s   | The patch does not contain any @author 
|      |                |            | tags.
|  +1  |    test4tests  |   0m  0s   | The patch appears to include 7 new or 
|      |                |            | modified test files.
|  +1  |    mvninstall  |  10m 38s   | HADOOP-13345 passed 
|  +1  |       compile  |   0m 29s   | HADOOP-13345 passed 
|  +1  |    checkstyle  |   0m 18s   | HADOOP-13345 passed 
|  +1  |       mvnsite  |   0m 30s   | HADOOP-13345 passed 
|  +1  |    mvneclipse  |   0m 22s   | HADOOP-13345 passed 
|  +1  |      findbugs  |   0m 43s   | HADOOP-13345 passed 
|  +1  |       javadoc  |   0m 15s   | HADOOP-13345 passed 
|  +1  |    mvninstall  |   0m 26s   | the patch passed 
|  +1  |       compile  |   0m 26s   | the patch passed 
|  +1  |         javac  |   0m 26s   | the patch passed 
|  +1  |    checkstyle  |   0m 15s   | the patch passed 
|  +1  |       mvnsite  |   0m 28s   | the patch passed 
|  +1  |    mvneclipse  |   0m 19s   | the patch passed 
|  +1  |    whitespace  |   0m  0s   | The patch has no whitespace issues. 
|  +1  |      findbugs  |   0m 48s   | the patch passed 
|  +1  |       javadoc  |   0m 11s   | the patch passed 
|  +1  |    asflicense  |   0m 12s   | The patch does not generate ASF License 
|      |                |            | warnings.{code}

> S3Guard: add delete tracking
> ----------------------------
>
>                 Key: HADOOP-13760
>                 URL: https://issues.apache.org/jira/browse/HADOOP-13760
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/s3
>            Reporter: Aaron Fabbri
>            Assignee: Sean Mackrory
>         Attachments: HADOOP-13760-HADOOP-13345.001.patch, 
> HADOOP-13760-HADOOP-13345.002.patch, HADOOP-13760-HADOOP-13345.003.patch, 
> HADOOP-13760-HADOOP-13345.004.patch, HADOOP-13760-HADOOP-13345.005.patch, 
> HADOOP-13760-HADOOP-13345.006.patch, HADOOP-13760-HADOOP-13345.007.patch, 
> HADOOP-13760-HADOOP-13345.008.patch, HADOOP-13760-HADOOP-13345.009.patch, 
> HADOOP-13760-HADOOP-13345.010.patch, HADOOP-13760-HADOOP-13345.011.patch, 
> HADOOP-13760-HADOOP-13345.012.patch
>
>
> Following the S3AFileSystem integration patch in HADOOP-13651, we need to add 
> delete tracking.
> Current behavior on delete is to remove the metadata from the MetadataStore.  
> To make deletes consistent, we need to add a {{isDeleted}} flag to 
> {{PathMetadata}} and check it when returning results from functions like 
> {{getFileStatus()}} and {{listStatus()}}.  In HADOOP-13651, I added TODO 
> comments in most of the places these new conditions are needed.  The work 
> does not look too bad.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to