Repository: phoenix Updated Branches: refs/heads/master 1f7a05b63 -> 5fecde8d8
PHOENIX-3195 Addendum. (James Taylor) Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/5fecde8d Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/5fecde8d Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/5fecde8d Branch: refs/heads/master Commit: 5fecde8d892fdc441aaa4e3ea8e195112da88705 Parents: 1f7a05b Author: Lars Hofhansl <[email protected]> Authored: Mon Aug 22 20:21:54 2016 -0700 Committer: Lars Hofhansl <[email protected]> Committed: Mon Aug 22 20:26:23 2016 -0700 ---------------------------------------------------------------------- .../java/org/apache/phoenix/compile/ExpressionCompiler.java | 6 +++++- .../main/java/org/apache/phoenix/compile/OrderByCompiler.java | 3 +++ 2 files changed, 8 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/phoenix/blob/5fecde8d/phoenix-core/src/main/java/org/apache/phoenix/compile/ExpressionCompiler.java ---------------------------------------------------------------------- diff --git a/phoenix-core/src/main/java/org/apache/phoenix/compile/ExpressionCompiler.java b/phoenix-core/src/main/java/org/apache/phoenix/compile/ExpressionCompiler.java index 1278494..0fd1876 100644 --- a/phoenix-core/src/main/java/org/apache/phoenix/compile/ExpressionCompiler.java +++ b/phoenix-core/src/main/java/org/apache/phoenix/compile/ExpressionCompiler.java @@ -393,6 +393,10 @@ public class ExpressionCompiler extends UnsupportedAllParseNodeVisitor<Expressio return ref; } + protected void addColumn(PColumn column) { + context.getScan().addColumn(column.getFamilyName().getBytes(), column.getName().getBytes()); + } + @Override public Expression visit(ColumnParseNode node) throws SQLException { ColumnRef ref = resolveColumn(node); @@ -407,7 +411,7 @@ public class ExpressionCompiler extends UnsupportedAllParseNodeVisitor<Expressio return LiteralExpression.newConstant(column.getDataType().toObject(ptr), column.getDataType()); } if (tableRef.equals(context.getCurrentTable()) && !SchemaUtil.isPKColumn(column)) { // project only kv columns - context.getScan().addColumn(column.getFamilyName().getBytes(), column.getName().getBytes()); + addColumn(column); } Expression expression = ref.newColumnExpression(node.isTableNameCaseSensitive(), node.isCaseSensitive()); Expression wrappedExpression = wrapGroupByExpression(expression); http://git-wip-us.apache.org/repos/asf/phoenix/blob/5fecde8d/phoenix-core/src/main/java/org/apache/phoenix/compile/OrderByCompiler.java ---------------------------------------------------------------------- diff --git a/phoenix-core/src/main/java/org/apache/phoenix/compile/OrderByCompiler.java b/phoenix-core/src/main/java/org/apache/phoenix/compile/OrderByCompiler.java index 6804375..9bc0c31 100644 --- a/phoenix-core/src/main/java/org/apache/phoenix/compile/OrderByCompiler.java +++ b/phoenix-core/src/main/java/org/apache/phoenix/compile/OrderByCompiler.java @@ -36,6 +36,7 @@ import org.apache.phoenix.parse.ParseNode; import org.apache.phoenix.parse.SelectStatement; import org.apache.phoenix.query.QueryServices; import org.apache.phoenix.query.QueryServicesOptions; +import org.apache.phoenix.schema.PColumn; import org.apache.phoenix.schema.PTableType; import org.apache.phoenix.schema.SortOrder; import org.apache.phoenix.schema.types.PInteger; @@ -97,6 +98,8 @@ public class OrderByCompiler { compiler = new ExpressionCompiler(context, GroupBy.EMPTY_GROUP_BY) { @Override protected Expression addExpression(Expression expression) {return expression;} + @Override + protected void addColumn(PColumn column) {} }; } else { compiler = new ExpressionCompiler(context, groupBy);
