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

Chris Nauroth commented on HADOOP-13179:
----------------------------------------

bq. Looking at the code, I don't think this actually fixes the problem of 
concurrent access to {{OptionsBuilder}}...there are lots of uses of the class 
in the Hadoop codebase, and they are all synchronized off different things.

That's correct.  The scope of this patch was limited to thread safety of 
{{GenericOptionsParser}}.

> 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