You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2022/06/30 07:27:46 UTC

[GitHub] [spark] MaxGekk commented on pull request #37017: [WIP][SPARK-39629][SQL] Support v2 SHOW FUNCTIONS

MaxGekk commented on PR #37017:
URL: https://github.com/apache/spark/pull/37017#issuecomment-1170865892

   > This PR only unifies the test, can we add the v2 show functions command?
   
   @cloud-fan I think mixing of tests unification and v2 impl in one PR is not good idea. I would prefer to unify tests first of all, and then try to implement v2. At the moment, I gathered tests only, and going to unify them to be able to run the same tests in v1 and v2 catalogs.
   


-- 
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.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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