[jira] [Created] (PHOENIX-6839) Rename ExecutableXXXStatement classes to CompilableXXXStatement to correctly name Compilable classes.

["Gourab Taparia (Jira)" <ji...@apache.org>]

Gourab Taparia created PHOENIX-6839:
---------------------------------------

             Summary: Rename ExecutableXXXStatement classes to CompilableXXXStatement to correctly name Compilable classes.
                 Key: PHOENIX-6839
                 URL: https://issues.apache.org/jira/browse/PHOENIX-6839
             Project: Phoenix
          Issue Type: Improvement
          Components: core
    Affects Versions: 5.1.2, 4.16.1, 4.14.1
            Reporter: Gourab Taparia


In the PhoenixStatement implementation there are various subclasses denoting various statements. For various implementations of CompilableStatements, it is seen that the underlying class is named prefixed with Executable (for e.g. ExecutableSelectStatement). This is confusing and makes the code readability harder. 



For e.g. Below is the declaration for ExecutableSelectStatement - [code|https://github.com/apache/phoenix/blob/master/phoenix-core/src/main/java/org/apache/phoenix/jdbc/PhoenixStatement.java#L665]

 
{code:java}
private static class ExecutableSelectStatement extends SelectStatement implements CompilableStatement {code}
 

This is there for other Statements too, ExecutableCloseStatement , ExecutableCreateTableStatement to name a few. We should fix all the related class declaration with this. 

 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)