[
https://issues.apache.org/jira/browse/HBASE-14127?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Appy updated HBASE-14127:
-------------------------
Status: Patch Available (was: Open)
> Refactor AssignmentManager.unassign() function which has lots of return
> statements
> ----------------------------------------------------------------------------------
>
> Key: HBASE-14127
> URL: https://issues.apache.org/jira/browse/HBASE-14127
> Project: HBase
> Issue Type: Improvement
> Reporter: Appy
> Assignee: Appy
> Priority: Minor
> Attachments: HBASE-14127-master-v2.patch, HBASE-14127-master.patch
>
>
> In AssignmentManager.unassign()
> * there are lots of return points
> * if condition is very non-trivial to understand
> * reuse of 'state' for different purpose makes things confusing
> A simple unfold of if-else with with better variable names should be
> sufficient.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
