You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "raulcd (via GitHub)" <gi...@apache.org> on 2024/10/10 13:03:25 UTC

[PR] GH-44369: [CI][Python] Remove ds requirement from test collection on test_dataset.py [arrow]

raulcd opened a new pull request, #44370:
URL: https://github.com/apache/arrow/pull/44370

   ### Rationale for this change
   
   If pyarrow has been built without dataset test collection errors due to pytest.mark.parametrize requiring dataset.
   
   ### What changes are included in this PR?
   
   Move dataset requirement outside of test collection by removing it from the parametrization.
   
   ### Are these changes tested?
   
   Via archery
   
   ### 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


Re: [PR] GH-44369: [CI][Python] Remove ds requirement from test collection on test_dataset.py [arrow]

Posted by "kou (via GitHub)" <gi...@apache.org>.
kou merged PR #44370:
URL: https://github.com/apache/arrow/pull/44370


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


Re: [PR] GH-44369: [CI][Python] Remove ds requirement from test collection on test_dataset.py [arrow]

Posted by "conbench-apache-arrow[bot] (via GitHub)" <gi...@apache.org>.
conbench-apache-arrow[bot] commented on PR #44370:
URL: https://github.com/apache/arrow/pull/44370#issuecomment-2406879984

   After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit e0d1f5105ba0dece2b74ed4a2e1530fccd0eef0d.
   
   There were no benchmark performance regressions. 🎉
   
   The [full Conbench report](https://github.com/apache/arrow/runs/31397498025) has more details. It also includes information about 1 possible false positive for unstable benchmarks that are known to sometimes produce them.


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


Re: [PR] GH-44369: [CI][Python] Remove ds requirement from test collection on test_dataset.py [arrow]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #44370:
URL: https://github.com/apache/arrow/pull/44370#issuecomment-2405043619

   Revision: d682c4f9382430d37b887b575ab943fba6929ea1
   
   Submitted crossbow builds: [ursacomputing/crossbow @ actions-b410c377ee](https://github.com/ursacomputing/crossbow/branches/all?query=actions-b410c377ee)
   
   |Task|Status|
   |----|------|
   |example-python-minimal-build-fedora-conda|[![GitHub Actions](https://github.com/ursacomputing/crossbow/actions/workflows/crossbow.yml/badge.svg?branch=actions-b410c377ee-github-example-python-minimal-build-fedora-conda)](https://github.com/ursacomputing/crossbow/actions/runs/11274638229/job/31354228054)|
   |example-python-minimal-build-ubuntu-venv|[![GitHub Actions](https://github.com/ursacomputing/crossbow/actions/workflows/crossbow.yml/badge.svg?branch=actions-b410c377ee-github-example-python-minimal-build-ubuntu-venv)](https://github.com/ursacomputing/crossbow/actions/runs/11274638284/job/31354228569)|


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


Re: [PR] GH-44369: [CI][Python] Remove ds requirement from test collection on test_dataset.py [arrow]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #44370:
URL: https://github.com/apache/arrow/pull/44370#issuecomment-2405038347

   :warning: GitHub issue #44369 **has been automatically assigned in GitHub** to PR creator.


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


Re: [PR] GH-44369: [CI][Python] Remove ds requirement from test collection on test_dataset.py [arrow]

Posted by "raulcd (via GitHub)" <gi...@apache.org>.
raulcd commented on PR #44370:
URL: https://github.com/apache/arrow/pull/44370#issuecomment-2405038460

   @github-actions crossbow submit example-python-minimal-build-*


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