MaxGekk commented on pull request #31675:
URL: https://github.com/apache/spark/pull/31675#issuecomment-788029728


   > Shall we simply make this fix for all the existing commands that have 
output in the constructor parameter (should be less than 5) without tests as 
the issue is so obvious?
   
   I would propose to have one test for at least the command `SHOW TABLES` (to 
prevent future problems while refactoring/fixing like 
https://issues.apache.org/jira/browse/SPARK-34156), and just fix other logical 
nodes w/o tests.


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

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