impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vuk Ercegovac (Code Review)" <>
Subject [Impala-ASF-CR] [DOCS] Tighten up advice about first COMPUTE INCREMENTAL STATS
Date Fri, 06 Oct 2017 17:11:28 GMT
Vuk Ercegovac has posted comments on this change. ( )

Change subject: [DOCS] Tighten up advice about first COMPUTE INCREMENTAL STATS

Patch Set 2:

File docs/shared/impala_common.xml:
PS2, Line 1228: DROP STATS</codeph> and
              :         <codeph>DROP INCREMENTAL STATS</codeph>)
are these drops required?
PS2, Line 1243:         be cached on every <cmdname>impalad</cmdname> host. If
this metadata for a table exceeds
more specifically, impalads that are also coordinators?
PS2, Line 1243: metadata for a table exceeds
              :         2 GB
is there a diagnostic page that we can point to here that explains how to find the size of
metadata (either via a sql query or a monitoring webpage)?
PS2, Line 1247: does not affect
does that mean lack of stats has not affect on optimization or something else?
File docs/topics/impala_partitioning.xml:
PS2, Line 611: frequently
PS2, Line 623: is a shortcut
I don't know what "shortcut" means here. I'd remove it.
File docs/topics/impala_perf_stats.xml:
PS2, Line 361: That situation is where you switch
I'd reword this part ("That situation is where ..."). Suggestion:

>From <keyword keyref="impala21_full"/> and higher, use the new feature to compute
statistics incrementally on just the partitions that changed. See ...
docs in impala_common mention "drop stats" before making a switch. that's not mentioned here.
what is the required/suggested usage? perhaps call this out as a "switch" and link to why
the user should avoid this.

To view, visit
To unsubscribe, visit

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia53a6518ce5541e5c9a2cd896856ce042a599b03
Gerrit-Change-Number: 7999
Gerrit-PatchSet: 2
Gerrit-Owner: John Russell <>
Gerrit-Reviewer: Alex Behm <>
Gerrit-Reviewer: Greg Rahn <>
Gerrit-Reviewer: John Russell <>
Gerrit-Reviewer: Mostafa Mokhtar <>
Gerrit-Reviewer: Silvius Rus <>
Gerrit-Reviewer: Vuk Ercegovac <>
Gerrit-Comment-Date: Fri, 06 Oct 2017 17:11:28 +0000
Gerrit-HasComments: Yes

  • Unnamed multipart/alternative (inline, 8-Bit, 0 bytes)
View raw message