Taewoo Kim has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1651
Change subject: Fix ASTERIXDB-1867: get Plan String only when context is not
null
......................................................................
Fix ASTERIXDB-1867: get Plan String only when context is not null
- Fix AbstractRuleController so that getPlanString() is
only executed when context is not null.
Change-Id: I4c4dfb50f16c48e4b09e8148d13f0debbd2de659
---
M
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/AbstractRuleController.java
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/51/1651/1
diff --git
a/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/AbstractRuleController.java
b/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/AbstractRuleController.java
index 98fe7b0..cfb3db1 100644
---
a/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/AbstractRuleController.java
+++
b/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/AbstractRuleController.java
@@ -66,7 +66,7 @@
}
private String getPlanString(Mutable<ILogicalOperator> opRef) throws
AlgebricksException {
- if (AlgebricksConfig.ALGEBRICKS_LOGGER.isLoggable(Level.FINE)) {
+ if (AlgebricksConfig.ALGEBRICKS_LOGGER.isLoggable(Level.FINE) &&
context != null) {
LogicalOperatorPrettyPrintVisitor pvisitor =
context.getPrettyPrintVisitor();
pvisitor.reset(new AlgebricksAppendable());
PlanPrettyPrinter.printOperator((AbstractLogicalOperator)
opRef.getValue(), pvisitor, 0);
--
To view, visit https://asterix-gerrit.ics.uci.edu/1651
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4c4dfb50f16c48e4b09e8148d13f0debbd2de659
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim <[email protected]>
