Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ARROW-14803: [R] Function not declared in scope #11772

Closed
wants to merge 3 commits into from

Conversation

jonkeane
Copy link
Member

@jonkeane jonkeane commented Nov 24, 2021

It looks like there was an order-of-imports issue and then one place where a return type wasn't quite right (when altrep wasn't defined). These changes make the old versions pass again (though there might be a better way to accomplish this!)

@github-actions
Copy link

@jonkeane
Copy link
Member Author

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

@jonkeane
Copy link
Member Author

@github-actions crossbow submit test-r-versions

@github-actions
Copy link

Revision: dfb6016

Submitted crossbow builds: ursacomputing/crossbow @ actions-1173

Task Status
test-r-versions Github Actions

@jonkeane
Copy link
Member Author

@github-actions crossbow submit test-r-versions

@github-actions
Copy link

Revision: 0b08b65

Submitted crossbow builds: ursacomputing/crossbow @ actions-1175

Task Status
test-r-versions Github Actions

@jonkeane
Copy link
Member Author

@github-actions crossbow submit test-r-versions

@github-actions
Copy link

Revision: a551447

Submitted crossbow builds: ursacomputing/crossbow @ actions-1178

Task Status
test-r-versions Github Actions

@@ -706,7 +706,7 @@ SEXP MakeAltrepVector(const std::shared_ptr<ChunkedArray>& chunked_array) {

bool is_arrow_altrep(SEXP) { return false; }

std::shared_ptr<Array> vec_to_arrow_altrep_bypass(SEXP x) { return nullptr; }
std::shared_ptr<ChunkedArray> vec_to_arrow_altrep_bypass(SEXP x) { return nullptr; }
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks odd that this goes from Array to ChunkedArray in this PR.

@thisisnic thisisnic closed this in 236362a Nov 29, 2021
@ursabot
Copy link

ursabot commented Nov 29, 2021

Benchmark runs are scheduled for baseline = 8581f36 and contender = 236362a. 236362a 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] ursa-i9-9960x
[Finished ⬇️0.18% ⬆️0.0%] 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

kou pushed a commit to kou/arrow that referenced this pull request Dec 1, 2021
It looks like there was an order-of-imports issue and then one place where a return type wasn't quite right (when altrep wasn't defined). These changes make the old versions pass again (though there might be a better way to accomplish this!)

Closes apache#11772 from jonkeane/r-versions-fix

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants