Skip to content

ARROW-16201: [R] SafeCallIntoR on 3.4#12887

Closed
jonkeane wants to merge 5 commits into
apache:masterfrom
jonkeane:ARROW-16201
Closed

ARROW-16201: [R] SafeCallIntoR on 3.4#12887
jonkeane wants to merge 5 commits into
apache:masterfrom
jonkeane:ARROW-16201

Conversation

@jonkeane

@jonkeane jonkeane commented Apr 14, 2022

Copy link
Copy Markdown
Member

Disabling the tests for now, 3.4 will no longer be in our support window shortly with the release of 4.2 also skip a number of tests that failed because of tzone being non-present on 3.4.

@jonkeane

Copy link
Copy Markdown
Member Author

@github-actions crossbow submit -g test-r-versions

@github-actions

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown
Invalid group(s) {'test-r-versions'}. Must be one of {'linux', 'verify-rc-source', 'cpp', 'verify-rc-jars', 'verify-rc', 'wheel', 'verify-rc-source-linux', 'python', 'r', 'example', 'vcpkg', 'linux-amd64', 'conda', 'homebrew', 'example-cpp', 'ruby', 'verify-rc-binaries', 'nightly-tests', 'test', 'linux-arm64', 'nightly-packaging', 'c-glib', 'integration', 'verify-rc-wheels', 'packaging', 'nightly-release', 'fuzz', 'nightly', 'verify-rc-source-macos'}
The Archery job run can be found at: https://github.com/apache/arrow/actions/runs/2168061224```

@jonkeane

Copy link
Copy Markdown
Member Author

@github-actions crossbow submit test-r-versions

@github-actions

Copy link
Copy Markdown

Revision: e20d179

Submitted crossbow builds: ursacomputing/crossbow @ actions-1857

Task Status
test-r-versions Github Actions

@jonkeane

Copy link
Copy Markdown
Member Author

@github-actions crossbow submit test-r-versions

@github-actions

Copy link
Copy Markdown

Revision: 975bdd5

Submitted crossbow builds: ursacomputing/crossbow @ actions-1860

Task Status
test-r-versions Github Actions

@jonkeane

Copy link
Copy Markdown
Member Author

@github-actions crossbow submit test-r-versions

@github-actions

Copy link
Copy Markdown

Revision: 0d5057f

Submitted crossbow builds: ursacomputing/crossbow @ actions-1861

Task Status
test-r-versions Github Actions

})

test_that("SafeCallIntoR errors from the non-R thread", {
skip_if_r_version("3.4.4")

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.

This should still get skipped, but I can also make sure that we error instead of crash #if !defined(HAS_UNWIND_PROTECT) (probably more important for #12323 since we'll be actually using it there). Actually, that PR is probably a better place to fix it (I'll make sure to rebase and run the test-r-versions on that PR on Monday).

@jonkeane jonkeane closed this in 63d2a9c Apr 14, 2022
@ursabot

ursabot commented Apr 16, 2022

Copy link
Copy Markdown

Benchmark runs are scheduled for baseline = 76e4f53 and contender = 63d2a9c. 63d2a9c 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
[Finished ⬇️0.08% ⬆️0.0%] test-mac-arm
[Failed ⬇️0.36% ⬆️0.0%] ursa-i9-9960x
[Finished ⬇️0.38% ⬆️0.0%] ursa-thinkcentre-m75q
Buildkite builds:
[Finished] <https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ec2-t3-xlarge-us-east-2/builds/518| 63d2a9c8 ec2-t3-xlarge-us-east-2>
[Finished] <https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-test-mac-arm/builds/505| 63d2a9c8 test-mac-arm>
[Failed] <https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-i9-9960x/builds/504| 63d2a9c8 ursa-i9-9960x>
[Finished] <https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-thinkcentre-m75q/builds/515| 63d2a9c8 ursa-thinkcentre-m75q>
[Finished] <https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ec2-t3-xlarge-us-east-2/builds/517| 76e4f536 ec2-t3-xlarge-us-east-2>
[Finished] <https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-test-mac-arm/builds/504| 76e4f536 test-mac-arm>
[Finished] <https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-i9-9960x/builds/503| 76e4f536 ursa-i9-9960x>
[Finished] <https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-thinkcentre-m75q/builds/514| 76e4f536 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants