You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by marmbrus <gi...@git.apache.org> on 2015/04/03 02:14:04 UTC

[GitHub] spark pull request: [SPARK-6034] [SQL] Support Describe Extended (...

Github user marmbrus commented on the pull request:

    https://github.com/apache/spark/pull/4784#issuecomment-89089368
  
    Thanks for working on this!  It would be great to have more uniform support here.  However, I wonder if the implementation can't be simpler?  Perhaps we could just add a `describe` method to catalog and then a simple `RunnableCommand` that just calls that.  This way even if we add other types of Catalogs, we don't have to add more commands.
    
    I propose we close this issue for now, and have a discussion about the schema for the values returned by `catalog.describe` on the JIRA.  Once we have agreement would be awesome if we can target this for 1.4.


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

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