Skip to content

ARROW-14515: [R] Add clang sanitizer to crossbow#11577

Closed
jonkeane wants to merge 5 commits intoapache:masterfrom
jonkeane:ARROW-14515-clang-san
Closed

ARROW-14515: [R] Add clang sanitizer to crossbow#11577
jonkeane wants to merge 5 commits intoapache:masterfrom
jonkeane:ARROW-14515-clang-san

Conversation

@jonkeane
Copy link
Copy Markdown
Member

@jonkeane jonkeane commented Oct 29, 2021

We should upstream r-fedora-clang-devel-san.dockerfile to rhub and use theirs when we do

@jonkeane
Copy link
Copy Markdown
Member Author

@github-actions
Copy link
Copy Markdown

@jonkeane
Copy link
Copy Markdown
Member Author

@github-actions crossbow submit test-fedora-r-clang-sanitizer

@github-actions
Copy link
Copy Markdown

Revision: 5da3fe8

Submitted crossbow builds: ursacomputing/crossbow @ actions-1060

Task Status
test-fedora-r-clang-sanitizer Azure

@jonkeane
Copy link
Copy Markdown
Member Author

jonkeane commented Nov 5, 2021

I'm going to merge this, ultimately we should upstream this, but I haven't had a chance to send the PR to r-hub yet, and I would like to make sure we have coverage for this sanitizer with 6.0.1 coming. https://issues.apache.org/jira/browse/ARROW-14617 for the upstreaming / moving this stuff around when we do.

@jonkeane jonkeane closed this in fe92778 Nov 5, 2021
@ursabot
Copy link
Copy Markdown

ursabot commented Nov 5, 2021

Benchmark runs are scheduled for baseline = e907a9f and contender = fe92778. fe92778 is a master commit associated with this PR. Results will be available as each benchmark for each run completes.
Conbench compare runs links:
[Finished ⬇️25.0% ⬆️0.0%] ec2-t3-xlarge-us-east-2
[Finished ⬇️0.0% ⬆️0.0%] ursa-i9-9960x
[Finished ⬇️0.36% ⬆️0.04%] ursa-thinkcentre-m75q
Supported benchmarks:
ursa-i9-9960x: langs = Python, R, JavaScript
ursa-thinkcentre-m75q: langs = C++, Java
ec2-t3-xlarge-us-east-2: cloud = True

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants