Jimexist commented on pull request #334:
URL: https://github.com/apache/arrow-datafusion/pull/334#issuecomment-845253719


   > Thanks @Jimexist -- I still think adding coverage for the cases I 
mentioned in 
https://github.com/apache/arrow-datafusion/pull/334/files#r636122381 would be 
good, but I am also comfortable with you doing so as a follow on pR.
   > 
   > Thanks again -- I vote to merge this on green CI
   
   yeah let me add test to sql.rs when the actual physical exec and result 
stream is done, in next PR


-- 
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:
us...@infra.apache.org


Reply via email to