You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@phoenix.apache.org by "Geoffrey Jacoby (JIRA)" <ji...@apache.org> on 2018/11/15 23:19:00 UTC

[jira] [Created] (PHOENIX-5023) Clean up giant anonymous inner classes

Geoffrey Jacoby created PHOENIX-5023:
----------------------------------------

             Summary: 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


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)