hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mingliang Liu (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HADOOP-13605) Clean up FileSystem javadocs, logging; improve diagnostics on FS load
Date Wed, 23 Nov 2016 23:49:58 GMT

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

Mingliang Liu commented on HADOOP-13605:

I like this patch, especially newly added javadocs. +1

> Clean up FileSystem javadocs, logging; improve diagnostics on FS load
> ---------------------------------------------------------------------
>                 Key: HADOOP-13605
>                 URL: https://issues.apache.org/jira/browse/HADOOP-13605
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>    Affects Versions: 2.8.0
>            Reporter: Steve Loughran
>            Assignee: Steve Loughran
>         Attachments: HADOOP-13605-004.patch, HADOOP-13605-005.patch, HADOOP-13605-branch-2-001.patch,
HADOOP-13605-branch-2-002.patch, HADOOP-13605-branch-2-003.patch
> We can't easily debug FS instantiation problems as there isn't much detail in what was
going on.
> We can add more logging, but cannot simply switch {{FileSystem.LOG}} to SLF4J —the
class is used in too many places, including tests which cast it. Instead, add a new private
SLF4J Logger, {{LOGGER}} and switch logging to it. 
> While working in the base FileSystem class, take the opportunity to clean up javadocs
and comments
> # add the list of exceptions, including indicating which base classes throw UnsupportedOperationExceptions
> # cut bits in the comments which are not true
> The outcome of this patch is that IDEs shouldn't highlight most of the file as flawed
in some way or another

This message was sent by Atlassian JIRA

To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org

View raw message