From aee9273a094a290104be0367e7cffad5fcd07b05 Mon Sep 17 00:00:00 2001 From: dataroaring <98214048+dataroaring@users.noreply.github.com> Date: Wed, 16 Feb 2022 11:57:12 +0800 Subject: [PATCH] [typo] translate comment in Chinese to English in SingleNodePlanner (#8038) --- .../java/org/apache/doris/planner/SingleNodePlanner.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/fe/fe-core/src/main/java/org/apache/doris/planner/SingleNodePlanner.java b/fe/fe-core/src/main/java/org/apache/doris/planner/SingleNodePlanner.java index 3b4dba71b4..b3aaf2af43 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/planner/SingleNodePlanner.java +++ b/fe/fe-core/src/main/java/org/apache/doris/planner/SingleNodePlanner.java @@ -711,8 +711,11 @@ public class SingleNodePlanner { candidates.add(new Pair<>(ref, new Long(materializedSize))); LOG.debug("The candidate of " + ref.getUniqueAlias() + ": " + materializedSize); } - // (ML): 这里感觉是不可能运行到的,因为起码第一个节点是inner join - if (candidates.isEmpty()) return null; + if (candidates.isEmpty()) { + // This branch should not be reached, because the first one should be inner join. + LOG.warn("Something wrong happens, the code should not be runned"); + return null; + } // order candidates by descending materialized size; we want to minimize the memory // consumption of the materialized hash tables required for the join sequence