asterixdb-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From wangs...@apache.org
Subject asterixdb git commit: Fix ASTERIXDB-1867: get Plan String only when context is not null
Date Sun, 02 Apr 2017 05:08:30 GMT
Repository: asterixdb
Updated Branches:
  refs/heads/master d66cfb8e5 -> ff915a9ec


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
Reviewed-on: https://asterix-gerrit.ics.uci.edu/1651
Reviewed-by: Xikui Wang <xkkwww@gmail.com>
Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>


Project: http://git-wip-us.apache.org/repos/asf/asterixdb/repo
Commit: http://git-wip-us.apache.org/repos/asf/asterixdb/commit/ff915a9e
Tree: http://git-wip-us.apache.org/repos/asf/asterixdb/tree/ff915a9e
Diff: http://git-wip-us.apache.org/repos/asf/asterixdb/diff/ff915a9e

Branch: refs/heads/master
Commit: ff915a9ec1b0abcb1084545ac2ba6bd4362e476b
Parents: d66cfb8
Author: Taewoo Kim <wangsaeu@yahoo.com>
Authored: Sat Apr 1 19:59:39 2017 -0700
Committer: Taewoo Kim <wangsaeu@gmail.com>
Committed: Sat Apr 1 22:08:11 2017 -0700

----------------------------------------------------------------------
 .../algebricks/core/rewriter/base/AbstractRuleController.java      | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/asterixdb/blob/ff915a9e/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/AbstractRuleController.java
----------------------------------------------------------------------
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 @@ public abstract class AbstractRuleController {
     }
 
     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);


Mime
View raw message