Skip to content

ARROW-9031: [R] Implement conversion from Type::UINT64 to R vector - #7455

Closed
nealrichardson wants to merge 2 commits into
apache:masterfrom
nealrichardson:r-uint64
Closed

ARROW-9031: [R] Implement conversion from Type::UINT64 to R vector#7455
nealrichardson wants to merge 2 commits into
apache:masterfrom
nealrichardson:r-uint64

Conversation

@nealrichardson

Copy link
Copy Markdown
Member

No description provided.

@github-actions

Copy link
Copy Markdown

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

Why does int64 use Converter_Int64 but the other types are handled differently?

Comment thread r/tests/testthat/test-Array.R Outdated

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.

Those tests are thorough!

@nealrichardson

Copy link
Copy Markdown
Member Author

Why does int64 use Converter_Int64 but the other types are handled differently?

Historical reasons I suppose. See also this discussion: #7441 (comment)

@nealrichardson

Copy link
Copy Markdown
Member Author

Deferring to https://jira.apache.org/jira/browse/ARROW-9083 questions of whether there are better ways to convert. This at least no longer errors and gives something usable.

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