Merge changelog and version from 54.3.1 into main#7340
Conversation
* bugfix: correct offsets when serializing a list of fixed sized list and non-zero start offset (#7318) * When serializing fixed length arrays, adjust the offsets for writing out * Add unit test * clippy warnings * Add unit test for nulls * Update unit test to account for which schema had nulls * Add missing type annotation (#7326) * Update version * Create changelog --------- Co-authored-by: Tim Saucer <timsaucer@gmail.com>
|
@alamb this is the requested PR to merge the changelog back into main |
| write_options, | ||
| )?; | ||
| return Ok(offset); | ||
| } else if let DataType::FixedSizeList(_, fixed_size) = data_type { |
There was a problem hiding this comment.
it is strange that github is showing this in the diff. However, I verified this code is already on main so this PR doesn't change the cide
arrow-rs/arrow-ipc/src/writer.rs
Lines 1730 to 1750 in a01886d
🤷
There was a problem hiding this comment.
And in fact the commit in main 0e73524 does not have these changes
There was a problem hiding this comment.
DOH! My original PR never was merged into main! It was approved, but not merged.
No it was merged I just doubled checked 😅 The fact it showed up here in this PR I think is some sort of github rendering artifact |

bugfix: correct offsets when serializing a list of fixed sized list and non-zero start offset (bugfix: correct offsets when serializing a list of fixed sized list and non-zero start offset #7318)
When serializing fixed length arrays, adjust the offsets for writing out
Add unit test
clippy warnings
Add unit test for nulls
Update unit test to account for which schema had nulls
Add missing type annotation (Add missing type annotation #7326)
Update version
Create changelog
Which issue does this PR close?
54.3.1(Mar 2025) #7330Rationale for this change
What changes are included in this PR?
Are there any user-facing changes?