Thomas Tauber-Marshall 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 4:

Also something I happened to notice that you could fix in this change - in 
fe/src/main/java/org/apache/impala/analysis/ShowStatsStmt.java line 102, 
there's an AnaysisException being thrown and the text of the exception is 
slightly wrong. It should say that SHOW PARTITIONS "must target an HDFS or Kudu 
table"


--
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: 4
Gerrit-Owner: Tamas Mate <tm...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Sahil Takiar <stak...@cloudera.com>
Gerrit-Reviewer: Tamas Mate <tm...@cloudera.com>
Gerrit-Reviewer: Thomas Tauber-Marshall <tmarsh...@cloudera.com>
Gerrit-Comment-Date: Fri, 28 Feb 2020 18:37:04 +0000
Gerrit-HasComments: No

Reply via email to