This is an automated email from the ASF dual-hosted git repository. stigahuang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/impala.git
commit 78339f122bcc85515311d2a4c66821b7e5095140 Author: Fang-Yu Rao <[email protected]> AuthorDate: Sun Oct 16 13:48:37 2022 -0700 IMPALA-11666: Modify the warning message when table statistics may be corrupt This patch revises the warning message that is output when 'hasCorruptTableStats_' of an HDFS table is set to true to let users know that it does not necessarily imply the existence of corrupt table statistics. Testing: - Verified that this patch passes the core tests in the DEBUG build. Change-Id: I404b00db97f1fa0f6e67995c6e85a124ccf242ef Reviewed-on: http://gerrit.cloudera.org:8080/19148 Reviewed-by: Quanlong Huang <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- fe/src/main/java/org/apache/impala/planner/Planner.java | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/fe/src/main/java/org/apache/impala/planner/Planner.java b/fe/src/main/java/org/apache/impala/planner/Planner.java index 45bb288e2..d34201156 100644 --- a/fe/src/main/java/org/apache/impala/planner/Planner.java +++ b/fe/src/main/java/org/apache/impala/planner/Planner.java @@ -329,8 +329,13 @@ public class Planner { tableNames.add(tableName.db_name + "." + tableName.table_name); } str.append( - "WARNING: The following tables have potentially corrupt table statistics.\n" + - "Drop and re-compute statistics to resolve this problem.\n" + + "The row count in one or more partitions in the following tables \n" + + "is either a) less than -1, or b) 0 but the size of all the files inside \n" + + "the partition(s) is positive.\n" + + "The latter case does not necessarily imply the existence of corrupt \n" + + "statistics when the corresponding tables are transactional.\n" + + "If it is suspected that there may be corrupt statistics, drop and \n" + + "re-compute statistics could resolve this problem.\n" + Joiner.on(", ").join(tableNames) + "\n"); hasHeader = true; }
