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

Todd Lipcon commented on HADOOP-6269:
-------------------------------------

bq. -1 tests included. The patch doesn't appear to include any new or modified 
tests.

There's no good way to regression test this since it's a very rare race. The 
patch is very small and should be hand verifiable, and correct operation of 
Configuration is ensured by hundreds of tests using it.

> Missing synchronization for defaultResources in Configuration.addResource
> -------------------------------------------------------------------------
>
>                 Key: HADOOP-6269
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6269
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: conf
>    Affects Versions: 0.20.1
>            Reporter: Todd Lipcon
>            Assignee: Todd Lipcon
>             Fix For: 0.20.2, 0.21.0
>
>         Attachments: hadoop-6269.txt
>
>
> Configuration.defaultResources is a simple ArrayList. In two places in 
> Configuration it is accessed without appropriate synchronization, which we've 
> seen to occasionally result in ConcurrentModificationExceptions.

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

Reply via email to