Till Westmann has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1114
Change subject: fix NPE in SqlppAstPrintVisitor
......................................................................
fix NPE in SqlppAstPrintVisitor
Change-Id: Ic5ac437366aa0eb2d60ceceeeba5b5f02d2946d2
---
M
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
1 file changed, 3 insertions(+), 2 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/14/1114/1
diff --git
a/asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
b/asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
index 908f799..3880fa1 100644
---
a/asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
+++
b/asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
@@ -132,9 +132,10 @@
public Void visit(Projection projection, Integer step) throws
AsterixException {
if (projection.star()) {
out.println(skip(step) + "*");
+ } else {
+ projection.getExpression().accept(this, step);
+ out.println(skip(step) + projection.getName());
}
- projection.getExpression().accept(this, step);
- out.println(skip(step) + projection.getName());
return null;
}
--
To view, visit https://asterix-gerrit.ics.uci.edu/1114
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic5ac437366aa0eb2d60ceceeeba5b5f02d2946d2
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <[email protected]>
