[
https://issues.apache.org/jira/browse/HBASE-10427?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13883413#comment-13883413
]
Sergey Shelukhin commented on HBASE-10427:
------------------------------------------
[~enis] [~nkeywal] fyi.
This patch (I am working on it now) should be simple.
> clean up HRegionLocation/ServerName usage
> ------------------------------------------
>
> Key: HBASE-10427
> URL: https://issues.apache.org/jira/browse/HBASE-10427
> Project: HBase
> Issue Type: Improvement
> Reporter: Sergey Shelukhin
> Assignee: Sergey Shelukhin
> Priority: Minor
>
> 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.
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)
