[Python] RecordBatch should fail gracefully on non-cpu devices #43727
Comments
danepitkin
changed the title
danepitkin
added a commit
to danepitkin/arrow
that referenced
this issue
Aug 16, 2024
danepitkin
added a commit
to danepitkin/arrow
that referenced
this issue
Aug 22, 2024
jorisvandenbossche
pushed a commit
that referenced
this issue
Sep 4, 2024
…43729) ### Rationale for this change Throw a python exception if a RecordBatch API isn't able to be used when the memory is backed by non-cpu devices. ### What changes are included in this PR? * Assert the device is CPU for APIs that only support CPU ### Are these changes tested? Pytests ### Are there any user-facing changes? The user experiences Python exceptions instead of segfaults for unsupported APIs. * GitHub Issue: #43727 Authored-by: Dane Pitkin <dpitkin@apache.org> Signed-off-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
|
Issue resolved by pull request 43729 |
zanmato1984
pushed a commit
to zanmato1984/arrow
that referenced
this issue
Sep 6, 2024
…ces (apache#43729) ### Rationale for this change Throw a python exception if a RecordBatch API isn't able to be used when the memory is backed by non-cpu devices. ### What changes are included in this PR? * Assert the device is CPU for APIs that only support CPU ### Are these changes tested? Pytests ### Are there any user-facing changes? The user experiences Python exceptions instead of segfaults for unsupported APIs. * GitHub Issue: apache#43727 Authored-by: Dane Pitkin <dpitkin@apache.org> Signed-off-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
Closed
4 tasks
khwilson
pushed a commit
to khwilson/arrow
that referenced
this issue
Sep 14, 2024
…ces (apache#43729) ### Rationale for this change Throw a python exception if a RecordBatch API isn't able to be used when the memory is backed by non-cpu devices. ### What changes are included in this PR? * Assert the device is CPU for APIs that only support CPU ### Are these changes tested? Pytests ### Are there any user-facing changes? The user experiences Python exceptions instead of segfaults for unsupported APIs. * GitHub Issue: apache#43727 Authored-by: Dane Pitkin <dpitkin@apache.org> Signed-off-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the enhancement requested
RecordBatch should throw Python exceptions instead of segfaults when trying to access non-cpu data from the cpu.
Component(s)
Python
The text was updated successfully, but these errors were encountered: