miklosgergely commented on a change in pull request #813: HIVE-22323 Fix Desc 
Table bugs
URL: https://github.com/apache/hive/pull/813#discussion_r334367697
 
 

 ##########
 File path: ql/src/test/results/clientpositive/alterColumnStats.q.out
 ##########
 @@ -144,15 +144,35 @@ PREHOOK: Input: default@p_n0
 POSTHOOK: query: desc formatted p_n0 c1
 POSTHOOK: type: DESCTABLE
 POSTHOOK: Input: default@p_n0
-# col_name             data_type               comment                         
                                                         
 
 Review comment:
   Yes, it was intentional. I think it is much better to have a consistent 
output format for `desc formatted <table> <column>` like commands, showing the 
same table all the time, explicitly showing that we don't have statistics for 
the column, instead of showing the smaller table as if a `desc <table> 
<column>` command would have been executed. The previous form is only 
implicitly showing that there are no statistics, which can be misleading.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org

Reply via email to