[
https://issues.apache.org/jira/browse/HADOOP-14041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15874906#comment-15874906
]
Aaron Fabbri commented on HADOOP-14041:
---------------------------------------
Thanks for the folow-up patch [~mackrorysd]. This is looking good. I'm
generally +1 on this but am attaching a patch that makes a couple of minor
changes.
- Remove a whitespace change.
- NullMetadataStore does support prune(), it is a no-op (matching with the rest
of that class).
- MetadataStoreTestBase tests the contract semantics (any files older than X
are removed), not the specific DynamoDBMetadataStore behavior of leaving
directories (that could be added in the TestDynamoDBMetadataStore subclass).
- Use allowMissing() in a finer-grained manner: we can still run the test and
assertNotCached() when allowMissing().
I tested the three MetadataStore integration tests in US West 2.
> CLI command to prune old metadata
> ---------------------------------
>
> Key: HADOOP-14041
> URL: https://issues.apache.org/jira/browse/HADOOP-14041
> Project: Hadoop Common
> Issue Type: Sub-task
> Components: fs/s3
> Reporter: Sean Mackrory
> Assignee: Sean Mackrory
> Attachments: HADOOP-14041-HADOOP-13345.001.patch,
> HADOOP-14041-HADOOP-13345.002.patch, HADOOP-14041-HADOOP-13345.003.patch,
> HADOOP-14041-HADOOP-13345.004.patch, HADOOP-14041-HADOOP-13345.005.patch,
> HADOOP-14041-HADOOP-13345.006.patch, HADOOP-14041-HADOOP-13345.007.patch
>
>
> Add a CLI command that allows users to specify an age at which to prune
> metadata that hasn't been modified for an extended period of time. Since the
> primary use-case targeted at the moment is list consistency, it would make
> sense (especially when authoritative=false) to prune metadata that is
> expected to have become consistent a long time ago.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
