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 [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
