Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/11193 )
Change subject: IMPALA-7425: Change incremental stats to pull from catalogd. ...................................................................... Patch Set 8: (2 comments) re-worked the part in compute stats to make use of the existing catalogd behavior that allows for the catalogd/impalad catalogs to get out of sync. that simplifies things. http://gerrit.cloudera.org:8080/#/c/11193/8/fe/src/main/java/org/apache/impala/analysis/ComputeStatsStmt.java File fe/src/main/java/org/apache/impala/analysis/ComputeStatsStmt.java: http://gerrit.cloudera.org:8080/#/c/11193/8/fe/src/main/java/org/apache/impala/analysis/ComputeStatsStmt.java@672 PS8, Line 672: for (Map.Entry<List<String>, TPartitionStats> stats : : re > re-working the checks here to probe from the local side (the remote already Done. http://gerrit.cloudera.org:8080/#/c/11193/8/fe/src/main/java/org/apache/impala/analysis/ComputeStatsStmt.java@683 PS8, Line 683: throw new AnalysisException( : "New > reconsidering this check and the one on L687. the prev. scheme with table v Done -- To view, visit http://gerrit.cloudera.org:8080/11193 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9d564808ca5157afe4e091909ca6cdac76e60d6e Gerrit-Change-Number: 11193 Gerrit-PatchSet: 8 Gerrit-Owner: Vuk Ercegovac <vercego...@cloudera.com> Gerrit-Reviewer: Bharath Vissapragada <bhara...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Tianyi Wang <tw...@cloudera.com> Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Vuk Ercegovac <vercego...@cloudera.com> Gerrit-Comment-Date: Fri, 31 Aug 2018 01:56:21 +0000 Gerrit-HasComments: Yes