Skip to content
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

feat: Implement IntoIterator for FixedSizeListArray #176

Merged
merged 2 commits into from
May 24, 2024

Conversation

dhruv9vats
Copy link
Contributor

@dhruv9vats dhruv9vats commented May 10, 2024

Implements IntoIterator for FixedSizeListArray by introducing a FixedSizeArrayChunk struct that yields an iterator with items being (fixed size) arrays of the supplied iterator's items.

@mbrobbel mbrobbel deleted the branch mbrobbel:main May 15, 2024 15:26
@mbrobbel mbrobbel closed this May 15, 2024
@mbrobbel mbrobbel reopened this May 15, 2024
@mbrobbel mbrobbel changed the base branch from arrow/union to main May 15, 2024 16:45
@mbrobbel mbrobbel force-pushed the fixed-size-list-into-iter branch from cff8d09 to 1e1f004 Compare May 15, 2024 16:47
@dhruv9vats dhruv9vats force-pushed the fixed-size-list-into-iter branch from 1e1f004 to 5530fff Compare May 16, 2024 12:54
@dhruv9vats dhruv9vats changed the title Implement IntoIter for FixedSizeListArray feat!: Implement IntoIter for FixedSizeListArray May 19, 2024
@dhruv9vats dhruv9vats force-pushed the fixed-size-list-into-iter branch 5 times, most recently from e387c1a to 3667169 Compare May 19, 2024 20:21
@dhruv9vats dhruv9vats force-pushed the fixed-size-list-into-iter branch 2 times, most recently from d6b33de to 1053044 Compare May 19, 2024 20:35
@dhruv9vats dhruv9vats changed the title feat!: Implement IntoIter for FixedSizeListArray feat!: Implement IntoIterator for FixedSizeListArray May 19, 2024
@dhruv9vats dhruv9vats force-pushed the fixed-size-list-into-iter branch 3 times, most recently from 36abc5c to 9c2d974 Compare May 19, 2024 21:29
@dhruv9vats dhruv9vats force-pushed the fixed-size-list-into-iter branch from 5bfe6a7 to cd8d65e Compare May 20, 2024 08:44
@dhruv9vats dhruv9vats force-pushed the fixed-size-list-into-iter branch from eb2f997 to a1a059f Compare May 21, 2024 11:52
@dhruv9vats dhruv9vats force-pushed the fixed-size-list-into-iter branch 5 times, most recently from dff0949 to fc9d3c4 Compare May 23, 2024 14:08
@dhruv9vats dhruv9vats marked this pull request as ready for review May 23, 2024 14:09
@dhruv9vats dhruv9vats changed the title feat!: Implement IntoIterator for FixedSizeListArray feat: Implement IntoIterator for FixedSizeListArray May 23, 2024
Copy link
Owner

@mbrobbel mbrobbel left a comment

Choose a reason for hiding this comment

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

Thanks @dhruv9vats!

src/arrow/array/fixed_size_list.rs Outdated Show resolved Hide resolved
Cargo.lock Outdated Show resolved Hide resolved
@dhruv9vats dhruv9vats force-pushed the fixed-size-list-into-iter branch from fc9d3c4 to 0605dc3 Compare May 23, 2024 14:48
@dhruv9vats dhruv9vats requested a review from mbrobbel May 23, 2024 14:49
@mbrobbel mbrobbel enabled auto-merge (squash) May 23, 2024 14:52
auto-merge was automatically disabled May 24, 2024 06:56

Head branch was pushed to by a user without write access

@dhruv9vats dhruv9vats force-pushed the fixed-size-list-into-iter branch from 0605dc3 to 4cab197 Compare May 24, 2024 06:56
Add a struct whose iterator yields rust arrays of fixed size (supplied
as a const generic `usize`)
Implement consuming `IntoIterator` for `FixedSizeListArray` by using
`FixedSizeArrayChunks` and update tests.
@dhruv9vats dhruv9vats force-pushed the fixed-size-list-into-iter branch from 4cab197 to 924a9df Compare May 24, 2024 06:58
@mbrobbel mbrobbel enabled auto-merge (squash) May 24, 2024 07:24
@mbrobbel mbrobbel merged commit a074bfe into mbrobbel:main May 24, 2024
14 checks passed
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