Skip to content

GH-15115: [R][CI] pyarrow tests fail on macos 10.13 due to missing pyarrow wheel - #15117

Merged
assignUser merged 2 commits into
apache:masterfrom
assignUser:fix-10.13-pyarrow
Jan 2, 2023
Merged

GH-15115: [R][CI] pyarrow tests fail on macos 10.13 due to missing pyarrow wheel#15117
assignUser merged 2 commits into
apache:masterfrom
assignUser:fix-10.13-pyarrow

Conversation

@assignUser

@assignUserassignUser commented Dec 29, 2022

Copy link
Copy Markdown
Member

@github-actions

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown

⚠️ GitHub issue #15115has been automatically assigned in GitHub to PR creator.

@assignUser

Copy link
Copy Markdown
MemberAuthor

@github-actions crossbow submit homebrew-r-autobrew

@github-actions

Copy link
Copy Markdown

Revision: cc42b4f

Submitted crossbow builds: ursacomputing/crossbow @ actions-e59dda9dc2

TaskStatus
homebrew-r-autobrewGithub Actions

@jonkeanejonkeane left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suspect this will still fail, but that might be the head issue we were seeing in the other PR.

Comment threadr/tests/testthat/test-python.R Outdated
# PyArrow doesn't support Python 3.6 or earlier
skip_on_python_older_than("3.7")
# no pyarrow wheels for macos 10.13
skip_if(Sys.info()["sysname"] == "Darwin" && Sys.info()["release"] < 14)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIUC the release here should actually be 18 (cf https://en.wikipedia.org/wiki/Darwin_(operating_system))

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh wow that versioning scheme is confusing :D

@assignUserassignUser changed the title GH-15115: [R][CI] pyarrow tests fail on macos 10.13 fue to missing pyarrow wheelGH-15115: [R][CI] pyarrow tests fail on macos 10.13 due to missing pyarrow wheelDec 29, 2022
@assignUser

Copy link
Copy Markdown
MemberAuthor

@github-actions crossbow submit homebrew-r-autobrew

and yes the head() issue will probably still pop up but the pyarrow tests should all be skipped like this (the first explicitly and the others due to missing pyarrow)

@github-actions

Copy link
Copy Markdown

Revision: b1411ce

Submitted crossbow builds: ursacomputing/crossbow @ actions-f64e192d54

TaskStatus
homebrew-r-autobrewGithub Actions

@assignUser
assignUser merged commit 1aa8f35 into apache:masterJan 2, 2023
@assignUser
assignUser deleted the fix-10.13-pyarrow branch January 2, 2023 15:21
@ursabot

Copy link
Copy Markdown

Benchmark runs are scheduled for baseline = 91e1a98 and contender = 1aa8f35. 1aa8f35 is a master commit associated with this PR. Results will be available as each benchmark for each run completes.
Conbench compare runs links:
[Finished ⬇️0.0% ⬆️0.0%] ec2-t3-xlarge-us-east-2
[Failed ⬇️1.11% ⬆️0.0%] test-mac-arm
[Finished ⬇️0.26% ⬆️0.0%] ursa-i9-9960x
[Failed ⬇️0.0% ⬆️0.0%] ursa-thinkcentre-m75q
Buildkite builds:
[Finished] 1aa8f355 ec2-t3-xlarge-us-east-2
[Failed] 1aa8f355 test-mac-arm
[Finished] 1aa8f355 ursa-i9-9960x
[Failed] 1aa8f355 ursa-thinkcentre-m75q
[Finished] 91e1a98c ec2-t3-xlarge-us-east-2
[Failed] 91e1a98c test-mac-arm
[Finished] 91e1a98c ursa-i9-9960x
[Failed] 91e1a98c ursa-thinkcentre-m75q
Supported benchmarks:
ec2-t3-xlarge-us-east-2: Supported benchmark langs: Python, R. Runs only benchmarks with cloud = True
test-mac-arm: Supported benchmark langs: C++, Python, R
ursa-i9-9960x: Supported benchmark langs: Python, R, JavaScript
ursa-thinkcentre-m75q: Supported benchmark langs: C++, Java

@ursabot

Copy link
Copy Markdown

['Python', 'R'] benchmarks have high level of regressions.
test-mac-arm

EpsilonPrime pushed a commit to EpsilonPrime/arrow that referenced this pull request Jan 5, 2023
…ing pyarrow wheel (apache#15117)
* Closes: apache#15115
Authored-by: Jacob Wujciak-Jens <jacob@wujciak.de>
Signed-off-by: Jacob Wujciak-Jens <jacob@wujciak.de>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[R][CI] pyarrow tests fail on macos 10.13 due to missing pyarrow wheel

3 participants

@assignUser@ursabot@jonkeane