Skip to content

MINOR: [R] switch on R_VERSION#43504

Merged
jonkeane merged 2 commits intoapache:mainfrom
jonkeane:r_existsVarInFrame
Aug 1, 2024
Merged

MINOR: [R] switch on R_VERSION#43504
jonkeane merged 2 commits intoapache:mainfrom
jonkeane:r_existsVarInFrame

Conversation

@jonkeane
Copy link
Copy Markdown
Member

@jonkeane jonkeane commented Aug 1, 2024

A follow on to #43243 #ifdef function declaration does not work, instead we must use the #if (R_VERSION >= R_Version(4, 2, 0)) pattern to only include code for specific versions.

Co-authored-by: Neal Richardson <neal.p.richardson@gmail.com>
@jonkeane jonkeane merged commit 0b9f06c into apache:main Aug 1, 2024
@jonkeane jonkeane removed the awaiting committer review Awaiting committer review label Aug 1, 2024
@jonkeane jonkeane deleted the r_existsVarInFrame branch August 1, 2024 21:55
@conbench-apache-arrow
Copy link
Copy Markdown

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

There were no benchmark performance regressions. 🎉

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

jonkeane added a commit that referenced this pull request Aug 5, 2024
A follow on to #43243 `#ifdef function declaration` does not work, instead we must use the `#if (R_VERSION >= R_Version(4, 2, 0))` pattern to only include code for specific versions.

Lead-authored-by: Jonathan Keane <jkeane@gmail.com>
Co-authored-by: Neal Richardson <neal.p.richardson@gmail.com>
Signed-off-by: Jonathan Keane <jkeane@gmail.com>
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.

2 participants