realno commented on pull request #1935: URL: https://github.com/apache/arrow-datafusion/pull/1935#issuecomment-1064562188
The PR looks good to me, thanks @yahoNanJing! One question I have is about testing, there will be a lot of major changes to Ballista, do we have good test coverage to ensure things are working as expected? Here is a related PR https://github.com/apache/arrow-datafusion/pull/1982, is Ballista integration test currently part of the pipeline? -- 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]
