Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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-3366: [R] Dockerfile for docker-compose setup #2770
ARROW-3366: [R] Dockerfile for docker-compose setup #2770
Changes from all commits
210a774
d8e55d7
4a7112b
94242b3
4d7f02c
64ea840
8a8376d
db9ee5b
f7c050e
0db3369
343456a
dd26d4b
9fcebe5
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@romainfrancois I believe this PR is now working! However, I want to talk about this test.
I had to remove this to get
R CMD CHECK
ordevtools::test()
to complete. I think the issue is thatexpect_error
cannot trap C++ exceptions. When running with this test restored, testing immediately stops with error:Have you ever seen that issue with
expect_error()
before? I tried googling around and nothing obvious jumped out to me. I have no idea how this could be working on Travis but not in my setup on docker :/.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Created issue for it: https://issues.apache.org/jira/browse/ARROW-3833
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another question is why an exception is being thrown at all
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That’s what Rcpp::stop does, but this is supposed to be caught by the generated code.