Skip to content

Make FixedSizedList work in group by - #11

Open
milevin wants to merge 1 commit into
sdf/54.2.1from
michael/fixed-sized-group-by
Open

Make FixedSizedList work in group by#11
milevin wants to merge 1 commit into
sdf/54.2.1from
michael/fixed-sized-group-by

Conversation

@milevin

@milevin milevin commented Jun 19, 2025

Copy link
Copy Markdown

No description provided.

@milevin
milevin requested a review from findepi June 19, 2025 06:12
@github-actions github-actions Bot added the arrow label Jun 19, 2025
findepi
findepi previously approved these changes Jun 23, 2025

@findepi findepi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

.

@findepi
findepi dismissed their stale review June 23, 2025 13:03

When combined with some tests (see apache#7705 for inspiration), the current implementation fails at decoding, so it likely doesn't solve the panic issue.

@findepi

findepi commented Jun 23, 2025

Copy link
Copy Markdown

Alternative: #12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants