Skip to content

Conversation

@cpcloud
Copy link
Contributor

@cpcloud cpcloud commented May 14, 2017

Negative indexing causes an interpreter segfault right now. After this PR, no segfaults, and reasonable error messages for out of bounds indexes.

Copy link
Member

@wesm wesm left a comment

Choose a reason for hiding this comment

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

+1. Thanks @cpcloud!

@asfgit asfgit closed this in edfb2dc May 15, 2017
jeffknupp pushed a commit to jeffknupp/arrow that referenced this pull request Jun 3, 2017
…rrow Table and Schema objects

Negative indexing causes an interpreter segfault right now. After this PR, no segfaults, and reasonable error messages for out of bounds indexes.

Author: Phillip Cloud <cpcloud@gmail.com>

Closes apache#686 from cpcloud/ARROW-1027 and squashes the following commits:

1bfcef4 [Phillip Cloud] ARROW-1027: [Python] Allow negative indexing in fields/columns on pyarrow Table and Schema objects
@cpcloud cpcloud deleted the ARROW-1027 branch June 23, 2017 04:25
pribor pushed a commit to GlobalWebIndex/arrow that referenced this pull request Oct 24, 2025
…e#686)

Bumps
[actions/download-artifact](https://github.com/actions/download-artifact)
from 4.1.9 to 4.2.1.

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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