[
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)