Github user skambha commented on the issue:

    https://github.com/apache/spark/pull/13822
  
    Thank you so much Andrew for reviewing.  I appreciate it. 
    I have taken care of your comments and also rebased.  Please take a look.  
Thanks. 
    
    My changes were doing two things: 
    1) Rename the output column for the SHOW PARTITIONS to be partition 
    2) Add an error message for non-existent table in Show Partitions command.
    and unit tests to cover both scenarios.
    
    But another [PR for 
SPARK-16543](https://github.com/apache/spark/commit/56183b84fb64ea13977d89ec55a9dd3997b4dacf)
 is already merged that takes care of (1) above.   There were no tests added in 
that one, so I am keeping the test to check (1) still here.   Is that ok?
    
    cc @yhuai, @cloud-fan, @andrewor14


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Reply via email to