Alex Behm has posted comments on this change. Change subject: IMPALA-3369: Add ALTER TABLE SET COLUMN STATS statement. ......................................................................
Patch Set 2: Dimitris, I agree that it is arguable what the right fix here is. Here are the reasons why I think the current solution is preferable: 1. A manual override for anything metadata/stats related is generally useful. 2. Fixing IMPALA-3562 is a very complex task if you think about dealing with incremental stats. You could be in a state where different partitions have partial stats for a different set of columns. I think the risk for that fix is pretty high. 3. Our compute stats does not compute some stats like numNulls. There simply is no way to set them. On the other hand, we also do not rely on them. -- To view, visit http://gerrit.cloudera.org:8080/3189 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I45cd8aa7241ea962788ba9ca7d0bbfd864c4304f Gerrit-PatchSet: 2 Gerrit-Project: Impala Gerrit-Branch: cdh5-2.6.0_5.8.0 Gerrit-Owner: Alex Behm <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Dimitris Tsirogiannis <[email protected]> Gerrit-HasComments: No
