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 2021/05/11 23:49:29 UTC

[GitHub] [arrow-datafusion] andygrove opened a new pull request #322: MINOR: Fix integration tests by adding datafusion-cli module to docker image

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


   The recent changes to move DataFusion CLI to a separate crate broke the Ballista integration tests. This PR simply adds this crate to the Docker image used by the integration tests.
   
   For anyone that is interested, integration tests can be run with this command:
   
   ```bash
   ./dev/integration-tests.sh
   ```
   
   I filed https://github.com/apache/arrow-datafusion/issues/321 for adding these tests to CI
   


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



[GitHub] [arrow-datafusion] alamb merged pull request #322: MINOR: Fix integration tests by adding datafusion-cli module to docker image

Posted by GitBox <gi...@apache.org>.
alamb merged pull request #322:
URL: https://github.com/apache/arrow-datafusion/pull/322


   


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



[GitHub] [arrow-datafusion] codecov-commenter commented on pull request #322: MINOR: Fix integration tests by adding datafusion-cli module to docker image

Posted by GitBox <gi...@apache.org>.
codecov-commenter commented on pull request #322:
URL: https://github.com/apache/arrow-datafusion/pull/322#issuecomment-839313152


   # [Codecov](https://codecov.io/gh/apache/arrow-datafusion/pull/322?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
   > Merging [#322](https://codecov.io/gh/apache/arrow-datafusion/pull/322?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (ca42d33) into [master](https://codecov.io/gh/apache/arrow-datafusion/commit/a1c6898f559020a4a56e301714cd16fd8229ca09?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (a1c6898) will **not change** coverage.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/arrow-datafusion/pull/322/graphs/tree.svg?width=650&height=150&src=pr&token=JXwWBKD3D9&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)](https://codecov.io/gh/apache/arrow-datafusion/pull/322?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   
   ```diff
   @@           Coverage Diff           @@
   ##           master     #322   +/-   ##
   =======================================
     Coverage   76.19%   76.19%           
   =======================================
     Files         141      141           
     Lines       23786    23786           
   =======================================
     Hits        18123    18123           
     Misses       5663     5663           
   ```
   
   
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/arrow-datafusion/pull/322?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/arrow-datafusion/pull/322?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Last update [a1c6898...ca42d33](https://codecov.io/gh/apache/arrow-datafusion/pull/322?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   


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