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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to