[
https://issues.apache.org/jira/browse/HADOOP-13171?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15310066#comment-15310066
]
Steve Loughran commented on HADOOP-13171:
-----------------------------------------
I'll see about a trunk patch too.
Checkstyle-wise, only one complaint is in production code
{code}
public S3AFastOutputStream(AmazonS3Client client,:10: More than 7 parameters
(found 9).
{code}
that's a wontfix. Test-code side; some on line-length and imports, which I'll
fix. A few on some fields for the test data structures where some of the fields
aren't encapsulated. I really think that bit of style checking in test code is
overkill. Production, I'm happy with it for the long-term maintenance,
especially for managing derived code. But for tests? Overkill.
Even so, those contract tests are externally used, so I'll do the wrapping. I
just think checkstyle is whining.
> Add StorageStatistics to S3A; instrument some more operations
> -------------------------------------------------------------
>
> Key: HADOOP-13171
> URL: https://issues.apache.org/jira/browse/HADOOP-13171
> Project: Hadoop Common
> Issue Type: Sub-task
> Components: fs/s3
> Affects Versions: 2.8.0
> Reporter: Steve Loughran
> Assignee: Steve Loughran
> Priority: Minor
> Attachments: HADOOP-13171-branch-2-001.patch,
> HADOOP-13171-branch-2-002.patch, HADOOP-13171-branch-2-003.patch,
> HADOOP-13171-branch-2-004.patch, HADOOP-13171-branch-2-005.patch,
> HADOOP-13171-branch-2-006.patch, HADOOP-13171-branch-2-007.patch,
> HADOOP-13171-branch-2-008.patch, HADOOP-13171-branch-2-009.patch,
> HADOOP-13171-branch-2-010.patch, HADOOP-13171-branch-2-011.patch,
> HADOOP-13171-branch-2-012.patch
>
>
> Add {{StorageStatistics}} support to S3A, collecting the same metrics as the
> instrumentation, but sharing across all instances.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
