alamb commented on issue #1453:
URL: 
https://github.com/apache/arrow-datafusion/issues/1453#issuecomment-998737056


   I was indeed thinking of 
https://github.com/apache/arrow-datafusion/issues/743 (specifically reorganize 
sql.rs so that it is not just a single giant module, and move code from 
context.rs there as well - item 4)
   
   Since we originally discussed 
https://github.com/apache/arrow-datafusion/issues/743, @Jimexist  added the 
postgres integration test which makes things a little more complicated.
   
   Perhaps the best thing to do is to start trying to describe what we have so 
far /  want the tests to look like. I started a PR to do so here 
https://github.com/apache/arrow-datafusion/pull/1471
   
   


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


Reply via email to