You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/02/03 21:48:09 UTC

[GitHub] [arrow-datafusion] alamb opened a new pull request #1742: Move optimize test out of context.rs

alamb opened a new pull request #1742:
URL: https://github.com/apache/arrow-datafusion/pull/1742


   # Which issue does this PR close?
   
   re  https://github.com/apache/arrow-datafusion/issues/743
   
   Follow on to https://github.com/apache/arrow-datafusion/pull/1684
   
    # Rationale for this change
   There are a bunch of tests in execution/context.rs that are not related to `ExecutionContext` (they are end to end tests for sql that happen to use the functions in `ExecutionContext`)
   
   This both slows down compile times in the `datafusion` crate as well as makes it harder to discover existing test coverage.
   
   This was annoying me recently when working in the `context` module so I decided to start moving the code over incrementally
   
   
   # What changes are included in this PR?
   Move a test
   
   # Are there any user-facing changes?
   No


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow-datafusion] alamb merged pull request #1742: Move optimize test out of context.rs

Posted by GitBox <gi...@apache.org>.

alamb merged pull request #1742:
URL: https://github.com/apache/arrow-datafusion/pull/1742


   


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org