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

Steve Loughran commented on HADOOP-13179:
-----------------------------------------

This just broke my never-gets-reviewed YARN-679 patch, where I'd allowed for 
services to be able to define their own options parser. Not your fault, just 
one of those details which arises when patches don't get reviewed in a timely 
manner. I'l have to fix my code there.

FWIW, I think we should jump to JCommander for future work: 
introspection+annotation based, much easier to work with.

> GenericOptionsParser is not thread-safe because commons-cli OptionBuilder is 
> not thread-safe
> --------------------------------------------------------------------------------------------
>
>                 Key: HADOOP-13179
>                 URL: https://issues.apache.org/jira/browse/HADOOP-13179
>             Project: Hadoop Common
>          Issue Type: Bug
>            Reporter: hongbin ma
>            Assignee: hongbin ma
>            Priority: Minor
>             Fix For: 2.8.0
>
>         Attachments: HADOOP-13179-master.patch, HADOOP-13179.001.patch
>
>
> I'm running into similar issues like 
> http://stackoverflow.com/questions/22462665/is-hadoops-toorunner-thread-safe, 
> the author's observation seem to make sense to me. However when I checked the 
> hadoop github trunk I found the issue still not fixed.
> Chris Nauroth further investigated this issue, here's his quote: 
> {quote}
> The root cause is that commons-cli OptionBuilder is not thread-safe.
> https://commons.apache.org/proper/commons-cli/apidocs/org/apache/commons/cl
> i/OptionBuilder.html
> According to this issue, commons-cli doesn't plan to change that and
> instead chose to document the lack of thread-safety.
> https://issues.apache.org/jira/browse/CLI-209
> I think we can solve this in Hadoop, probably with a one-line change to
> make GenericOptionsParser#buildGeneralOptions a synchronized method.
> {quote}
> I'll soon upload a patch for this



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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