[ 
https://issues.apache.org/jira/browse/PHOENIX-5023?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Thomas D'Silva updated PHOENIX-5023:
------------------------------------
    Summary: [UMBRELLA] Clean up giant anonymous inner classes  (was: Clean up 
giant anonymous inner classes)

> [UMBRELLA] Clean up giant anonymous inner classes
> -------------------------------------------------
>
>                 Key: PHOENIX-5023
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-5023
>             Project: Phoenix
>          Issue Type: Improvement
>            Reporter: Geoffrey Jacoby
>            Priority: Minor
>
> A frequent anti-pattern in the Phoenix code base is the use of huge anonymous 
> inner classes, often in the midst of functions that are already very large, 
> or even the parameter list of constructor calls. (The latter means that we 
> have single statements that are technically hundreds of lines long!)  
> This makes the logic difficult to follow and prevents abstraction. Modern 
> IDEs make changing these to named inner classes quite simple. 
> This is intended as an umbrella item with subtasks refactoring particular 
> classes, which can be added to over time. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to