[ 
https://issues.apache.org/jira/browse/HADOOP-13998?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Steve Loughran updated HADOOP-13998:
------------------------------------
    Attachment: HADOOP-13998-003.patch

HADOOP-13998 patch 003
* checkstyle issues
* a bit more IDE cleanup (inc /** -> /* in top comment)
* TestPathMetadataDynamoDBTranslation -> callable from lambda
* TestS3GuardConcurrentOps made java 7 friendly

My IDE was confused and thought it was Java 7, which helped find a couple of 
java 8 bits of the test. Fixed them for ease of backporting this to 2.9.

Identified a couple of issues we should look at/clarify

# {{LocalMetadataStore.prune()}} is modifying the iterator of the dirhash 
during the iteration using put(). Is it safe to do that?. It may be better to 
build the list of entries to add, and do it after that initial iteration

# DynamoDbClientFactory should be able to pick up StringUtils.join, either the 
hadoop one or one of the commons-lang ones.


> Merge initial s3guard release into trunk
> ----------------------------------------
>
>                 Key: HADOOP-13998
>                 URL: https://issues.apache.org/jira/browse/HADOOP-13998
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/s3
>    Affects Versions: 3.0.0-beta1
>            Reporter: Steve Loughran
>            Assignee: Steve Loughran
>         Attachments: HADOOP-13998-001.patch, HADOOP-13998-002.patch, 
> HADOOP-13998-003.patch
>
>
> JIRA to link in all the things we think are needed for a preview/merge into 
> trunk



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