Skip to content

ARROW-6775: [C++][Python] Implement list_value_lengths and list_parent_indices functions #7632

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.

Sign up for GitHub

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

Jump to bottom
Closed
wesm wants to merge 5 commits into from
Closed

ARROW-6775: [C++][Python] Implement list_value_lengths and list_parent_indices functions #7632

wesm wants to merge 5 commits into from

Conversation

@wesm
Copy link
Member

@wesm wesm commented Jul 3, 2020

This adds two functions that operate on list types:

  • list_value_lengths: returns an int32 (for List) or int64 (for LargeList) array with the number of elements in each list value slot
  • list_parent_indices: returns an int32/int64 array with the same length as the child values array of a List type where each value is the index of the list "slot" containing each child value

@wesm
Copy link
Member Author

wesm commented Jul 3, 2020

cc @brills

@github-actions
Copy link

github-actions bot commented Jul 3, 2020

https://issues.apache.org/jira/browse/ARROW-6775

@wesm wesm force-pushed the some-list-functions branch from 984e35c to d5795a5 Compare July 5, 2020 00:54
@wesm
Copy link
Member Author

wesm commented Jul 5, 2020

CI is OK now, this just needs to be reviewed

@wesm
Copy link
Member Author

wesm commented Jul 6, 2020

Any opinions about these new APIs? @xhochy or @pitrou could you take a quick look?

@pitrou pitrou force-pushed the some-list-functions branch from d5795a5 to aacb398 Compare July 7, 2020 09:27
@pitrou pitrou closed this in e1c3334 Jul 7, 2020
@brills
Copy link
Contributor

brills commented Jul 8, 2020

@wesm: Thanks! These will replace our own implementations.

@wesm wesm deleted the some-list-functions branch July 8, 2020 21:18
@asfimport asfimport mentioned this pull request Jul 8, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pitrou pitrou pitrou approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@wesm @brills @pitrou