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

clean up HRegionLocation/ServerName usage

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.99.0
    • Component/s: None
    • Labels:
      None

      Description

      I noticed that AsyncProcess updates cache location on failures using a single HRL object that is key to the map that is intended to be by server and contains requests for multiple regions (i.e. MultiAction contains requests for regions A, B, C, and sits in a map by HRL with HRL from server A as a key; in case of failure for e.g. request to B, or entire multiaction, the location from map key will be passed to updateCache... methods, even though it's not for the correct region). It may cause some subtle mistakes in cache updates.
      I think it'd be good to clean up HRL usage around AP and other classes - if we intend to have a server name, then we should use ServerName not HRL.

        Attachments

        1. HBASE-10427.01.patch
          37 kB
          Sergey Shelukhin
        2. HBASE-10427.patch
          32 kB
          Sergey Shelukhin

          Activity

            People

            • Assignee:
              sershe Sergey Shelukhin
              Reporter:
              sershe Sergey Shelukhin
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: