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

Chinmay Kulkarni updated PHOENIX-5023:
--------------------------------------
    Labels: phoenix-hardening  (was: )

> [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
>              Labels: phoenix-hardening
>
> 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
(v8.3.2#803003)

Reply via email to