Repository: kylin Updated Branches: refs/heads/1.x-staging 432e6103f -> e9afc45d3
KYLIN-993 Remove unnecessary logs Project: http://git-wip-us.apache.org/repos/asf/kylin/repo Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/e9afc45d Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/e9afc45d Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/e9afc45d Branch: refs/heads/1.x-staging Commit: e9afc45d3fa37ca9077c9f45147434a9e354e7ad Parents: 432e610 Author: lidongsjtu <don...@ebay.com> Authored: Mon Jan 11 10:16:02 2016 +0800 Committer: lidongsjtu <don...@ebay.com> Committed: Mon Jan 11 10:16:02 2016 +0800 ---------------------------------------------------------------------- .../coprocessor/CoprocessorTupleFilterTranslator.java | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/kylin/blob/e9afc45d/storage/src/main/java/org/apache/kylin/storage/hbase/coprocessor/CoprocessorTupleFilterTranslator.java ---------------------------------------------------------------------- diff --git a/storage/src/main/java/org/apache/kylin/storage/hbase/coprocessor/CoprocessorTupleFilterTranslator.java b/storage/src/main/java/org/apache/kylin/storage/hbase/coprocessor/CoprocessorTupleFilterTranslator.java index bd2b912..6f1c964 100644 --- a/storage/src/main/java/org/apache/kylin/storage/hbase/coprocessor/CoprocessorTupleFilterTranslator.java +++ b/storage/src/main/java/org/apache/kylin/storage/hbase/coprocessor/CoprocessorTupleFilterTranslator.java @@ -49,13 +49,15 @@ public class CoprocessorTupleFilterTranslator implements ITupleFilterTranslator public TupleFilter translate(TupleFilter tupleFilter) { TupleFilter translated = null; if (tupleFilter instanceof CompareTupleFilter) { - logger.info("Translation to IN clause: " + tupleFilter); translated = translateCompareTupleFilter((CompareTupleFilter) tupleFilter); - logger.info(translated == null ? "Failed, will use Calcite to handle computed comparison." : "Succeed: " + translated); + if (translated != null) { + logger.info("Translated {" + tupleFilter + "} to IN clause: {" + translated + "}"); + } } else if (tupleFilter instanceof FunctionTupleFilter) { - logger.info("Translation to IN clause: " + tupleFilter); translated = translateFunctionTupleFilter((FunctionTupleFilter) tupleFilter); - logger.info(translated == null ? "Failed, will use Calcite to handle computed column." : "Succeed: " + translated); + if (translated != null) { + logger.info("Translated {" + tupleFilter + "} to IN clause: {" + translated + "}"); + } } return translated == null ? tupleFilter : translated; }
