Skip to content

GH-49578: [CI][R] gcc sanitizer failure - #49581

Merged
thisisnic merged 1 commit into
apache:mainfrom
jonkeane:stringi_san_skip
Mar 24, 2026
Merged

thisisnic merged 1 commit into
apache:mainfrom
jonkeane:stringi_san_skip

Conversation

@jonkeane

@jonkeane jonkeane commented Mar 23, 2026 •

Copy link
Copy Markdown
Member

Rationale for this change

Resolve the crossbow failure, I've already created an issue upstream

What changes are included in this PR?

Skipping a test under sanitization

Are these changes tested?

They are the tests

Are there any user-facing changes?

No

@jonkeane
jonkeane requested a review from thisisnic as a code owner March 23, 2026 12:46
@jonkeane

Copy link
Copy Markdown
Member Author

@github-actions crossbow submit test-r-linux-sanitizers

@github-actions

Copy link
Copy Markdown

⚠️ GitHub issue #49578 has no components, please add labels for components.

@github-actions

Copy link
Copy Markdown

Revision: 570e975

Submitted crossbow builds: ursacomputing/crossbow @ actions-c0b45c0cb0

Task Status
test-r-linux-sanitizers GitHub Actions

@amoeba amoeba 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.

+1

@thisisnic
thisisnic merged commit a942ed6 into apache:main Mar 24, 2026
16 of 17 checks passed
@thisisnic thisisnic removed the awaiting committer review Awaiting committer review label Mar 24, 2026
@github-actions github-actions Bot added the awaiting merge Awaiting merge label Mar 24, 2026
thisisnic pushed a commit to thisisnic/arrow that referenced this pull request Mar 24, 2026
### Rationale for this change

Resolve the crossbow failure, I've already created an issue upstream

### What changes are included in this PR?

Skipping a test under sanitization

### Are these changes tested?

They are the tests

### Are there any user-facing changes?

No
* GitHub Issue: apache#49578

Authored-by: Jonathan Keane <jkeane@gmail.com>
Signed-off-by: Nic Crane <thisisnic@gmail.com>
@conbench-apache-arrow

Copy link
Copy Markdown

After merging your PR, Conbench analyzed the 3 benchmarking runs that have been run so far on merge-commit a942ed6.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 13 possible false positives for unstable benchmarks that are known to sometimes produce them.

thisisnic pushed a commit to thisisnic/arrow that referenced this pull request Apr 6, 2026
### Rationale for this change

Resolve the crossbow failure, I've already created an issue upstream

### What changes are included in this PR?

Skipping a test under sanitization

### Are these changes tested?

They are the tests

### Are there any user-facing changes?

No
* GitHub Issue: apache#49578

Authored-by: Jonathan Keane <jkeane@gmail.com>
Signed-off-by: Nic Crane <thisisnic@gmail.com>
Mottl pushed a commit to Mottl/arrow that referenced this pull request May 26, 2026
### Rationale for this change

Resolve the crossbow failure, I've already created an issue upstream

### What changes are included in this PR?

Skipping a test under sanitization

### Are these changes tested?

They are the tests

### Are there any user-facing changes?

No
* GitHub Issue: apache#49578

Authored-by: Jonathan Keane <jkeane@gmail.com>
Signed-off-by: Nic Crane <thisisnic@gmail.com>
Sign up for free to 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.

3 participants