tustvold opened a new pull request #1246:
URL: https://github.com/apache/arrow-rs/pull/1246
# Which issue does this PR close?
Closes #1245.
# Rationale for this change
Originally reported by @bjchambers, a sanity check added in #1054 fires when
reading NullArrays.
This highlighted two problems:
* `NullArrayReader` has always been somewhat broken as it would never flush
the accumulated null buffer, instead just growing it indefinitely
* There were literally no tests for NullArrayReader :scream:
# What changes are included in this PR?
Fixes the panic, and adds a basic test of the `NullArrayReader`
# Are there any user-facing changes?
NullArrayReader should work again
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
