Sahil Takiar has posted comments on this change. ( http://gerrit.cloudera.org:8080/15199 )
Change subject: IMPALA-5308: Resolve confusing Kudu SHOW TABLE STATS output ...................................................................... Patch Set 2: IMO it would be better to format the output of 'show table stats [impala-kudu-table-name]' as follows: +-----------------+ | # Rows | +-----------------+ | [total-num-rows] | +-----------------+ Having the partition information with 'NA' can be confusing, it makes it seem like there is a bug in Impala or Kudu that is causing the stats to not be available. Plus it doesn't add any extra information. Added Thomas to the review to see what he thinks as well. -- To view, visit http://gerrit.cloudera.org:8080/15199 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ice4b8df65f0a53fe14b8fbe35d82c9887ab9a041 Gerrit-Change-Number: 15199 Gerrit-PatchSet: 2 Gerrit-Owner: Tamas Mate <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Sahil Takiar <[email protected]> Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]> Gerrit-Comment-Date: Thu, 13 Feb 2020 17:45:34 +0000 Gerrit-HasComments: No
