hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Luke Lu (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HADOOP-6787) Factor out glob pattern code from FileContext and Filesystem
Date Tue, 01 Jun 2010 23:51:40 GMT

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

Luke Lu commented on HADOOP-6787:
---------------------------------

Thanks for the +1!

Most IDEs would indicate (in my case, with a green circle with I in it) an implementing method.
Clicking or hovering on the indicator would show you the  javadoc of the interface method.
I personally prefer keeping the javadocs in interfaces/abstract classes and not duplicating
them in implementing classes.   I hope that it's not a bug in Hudson's javadoc tool to +1
my patch as well :)

> Factor out glob pattern code from FileContext and Filesystem
> ------------------------------------------------------------
>
>                 Key: HADOOP-6787
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6787
>             Project: Hadoop Common
>          Issue Type: Improvement
>            Reporter: Luke Lu
>            Assignee: Luke Lu
>             Fix For: 0.22.0
>
>         Attachments: hadoop-6787-trunk-v1.patch, hadoop-6787-y20s-v1.patch
>
>
> The GlobFilter class (especially the error-prone setRegex method) is duplicated in both
FileContext and Filesystem. And I'd like to be able to reuse the most of the setRegex code
for glob pattern matching in metrics filter configurations.
> We could at least factor out the hairy setRegex code and package them into a GlobPattern
class with a minimum compile method.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message