Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 15FE7200C58 for ; Sun, 2 Apr 2017 07:08:32 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 15050160BA0; Sun, 2 Apr 2017 05:08:32 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 5B957160B9D for ; Sun, 2 Apr 2017 07:08:31 +0200 (CEST) Received: (qmail 47335 invoked by uid 500); 2 Apr 2017 05:08:30 -0000 Mailing-List: contact commits-help@asterixdb.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@asterixdb.apache.org Delivered-To: mailing list commits@asterixdb.apache.org Received: (qmail 47326 invoked by uid 99); 2 Apr 2017 05:08:30 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 02 Apr 2017 05:08:30 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 2A9E2DFE59; Sun, 2 Apr 2017 05:08:30 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: wangsaeu@apache.org To: commits@asterixdb.apache.org Message-Id: X-Mailer: ASF-Git Admin Mailer Subject: asterixdb git commit: Fix ASTERIXDB-1867: get Plan String only when context is not null Date: Sun, 2 Apr 2017 05:08:30 +0000 (UTC) archived-at: Sun, 02 Apr 2017 05:08:32 -0000 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 Sonar-Qube: Jenkins Tested-by: Jenkins BAD: Jenkins Integration-Tests: Jenkins 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 Authored: Sat Apr 1 19:59:39 2017 -0700 Committer: Taewoo Kim 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 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);