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

ramkrishna.s.vasudevan edited comment on HBASE-6089 at 5/30/12 10:09 AM:
-------------------------------------------------------------------------

I was not clear in the previous comment.  Thinking again on this, there are 
some places where we wait for the this.regions to be populated.
{code}
    synchronized(regions) {
      while(!regions.containsKey(regionInfo)) {
        // We should receive a notification, but it's
        //  better to have a timeout to recheck the condition here:
        //  it lowers the impact of a race condition if any
        regions.wait(100);
      }
{code}
When we need to update the this.servers also along with this.regions then we 
need to have some sync block.  I agree with Anoop here.
                
      was (Author: ram_krish):
    Thinking again on this, there are some places where we wait for the 
this.regions to be populated.
{code}
{code}
                  
> SSH and AM.joinCluster causes Concurrent Modification exception.
> ----------------------------------------------------------------
>
>                 Key: HBASE-6089
>                 URL: https://issues.apache.org/jira/browse/HBASE-6089
>             Project: HBase
>          Issue Type: Bug
>    Affects Versions: 0.92.1, 0.94.0
>            Reporter: ramkrishna.s.vasudevan
>            Assignee: rajeshbabu
>             Fix For: 0.90.7, 0.92.2, 0.96.0, 0.94.1
>
>         Attachments: HBASE-6089_92.patch, HBASE-6089_94.patch, 
> HBASE-6089_trunk.patch
>
>
> AM.regions map is parallely accessed in SSH and Master initialization leading 
> to ConcurrentModificationException.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to