Github user jaanai0 commented on a diff in the pull request:
https://github.com/apache/phoenix/pull/355#discussion_r226303373
--- Diff:
phoenix-core/src/main/java/org/apache/phoenix/jdbc/PhoenixStatement.java ---
@@ -1435,6 +1384,32 @@ public MutationState execute() throws SQLException {
}
}
+ private static class ExecutableModifyColumnStatement extends
ModifyColumnStatement implements CompilableStatement {
+
+ ExecutableModifyColumnStatement(NamedTableNode table, PTableType
tableType, ColumnDef columnDef, boolean ifNotExists) {
+ super(table, tableType, columnDef, ifNotExists);
+ }
+
+ @SuppressWarnings("unchecked")
+ @Override
+ public MutationPlan compilePlan(final PhoenixStatement stmt,
Sequence.ValueOp seqAction) throws SQLException {
+ final StatementContext context = new StatementContext(stmt);
+ return new BaseMutationPlan(context, this.getOperation()) {
+
+ @Override
--- End diff --
Ok, I will add a test for the explain plan
---
