Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-17653

HBASE-17624 rsgroup synchronizations will (distributed) deadlock

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 2.0.0
    • rsgroup
    • None
    • Reviewed

    Description

      Follow-on from HBASE-17624. HBASE-17624 made it so one thread only has access to the rsgroup administrator. In tail of HBASE-17624 toffer describes scenario under which we may end up in a deadlock (distributed). Let me repeat toffer comment...

      Both read/write access can't be single threaded. Consider the situation:
      1. move_rsgroup_servers is called
      2. while #1 is happening rsgroup region is in transition (rpc thread in #1 holds monitor lock)
      3. while #2 is happening meta is in transition.
      Balancer tries to figure out plan for meta region tries to get monitor lock but can't. rpc thread task won't release monitor lock since rsgroup region never gets assigned. rsgroup region never gets assigned because it can't update meta with new state.
      There's a good chance this can be reproduce just by moving both rsgroup and meta region onto the same RS and call move_rsgoup_servers on the same RS.
      A bunch different actors will query from group affiliation so we can't have writes block reads.
      
      ....
      
      In the code prior to this patch the getter methods that retrieve group information (getRSGroup, ofTable, OfServer, etc) don't require the monitor lock so the deadlock cycle is broken.
      
      
      ....
      
      The methods that does mutations and updates to zk and hbase:rsgroup are synchronized appropriately. Point me to where the incoherence is?
      

      This issue is about testing/fixing/restoring rsgroup access. Will be back.

      Attachments

        1. HBASE-17653.master.003.patch
          29 kB
          Michael Stack
        2. HBASE-17653.master.002.patch
          29 kB
          Michael Stack
        3. HBASE-17653.master.001.patch
          24 kB
          Michael Stack

        Issue Links

          Activity

            People

              stack Michael Stack
              stack Michael Stack
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: