Github user dilipbiswal commented on the issue:

    https://github.com/apache/spark/pull/15958
  
    @gatorsmile Thank you for cc'ing me. Overall it looks good to me. 
    I have a few observations/comments that i would list below which hopefully 
should help us
    decide.
    
    1) Hive actually has two commands i) show tables ii) show table and the 
extended
       switch is applicable for the "show table" command. Have we decided to 
merge
       the semantics of these two commands in an effort to similify ?
    
    2) As a side effect, we actually print the database name and table name two 
times. 
       Is that ok ?
    
    3) The format of hive seems better as there is a newline between two 
tables. I created
       a regular table and a temporary view with same prefix and tried the 
command. Its very
       difficult to distinguish the output between table and view. here is how 
it looks.
    
    ```SQL
    spark-sql> create table realtable(c1 int) using parquet;
    Time taken: 0.467 seconds
    spark-sql> create temporary view realview (c1 int, c2 int) using parquet;
    Time taken: 0.053 seconds
    spark-sql> show tables extended like 'real*';
    default     realtable       false   CatalogTable(
        Table: `default`.`realtable`
        Owner: cloudera
        Created: Tue Nov 22 17:41:56 PST 2016
        Last Access: Wed Dec 31 16:00:00 PST 1969
        Type: MANAGED
        Schema: [StructField(c1,IntegerType,true)]
        Provider: parquet
        Properties: [transient_lastDdlTime=1479865316]
        Storage(Location: file:/user/hive/warehouse/realtable, InputFormat: 
org.apache.hadoop.hive.ql.io.parquet.MapredParquetInputFormat, OutputFormat: 
org.apache.hadoop.hive.ql.io.parquet.MapredParquetOutputFormat, Serde: 
org.apache.hadoop.hive.ql.io.parquet.serde.ParquetHiveSerDe, Properties: 
[serialization.format=1]))
        realview        true    CatalogTable(
        Table: `realview`
        Created: Tue Nov 22 17:42:28 PST 2016
        Last Access: Wed Dec 31 15:59:59 PST 1969
        Type: VIEW
        Schema: [StructField(c1,IntegerType,true), 
StructField(c2,IntegerType,true)]
        Storage())
    Time taken: 0.104 seconds, Fetched 2 row(s)
    
    ```


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to