rluvaton opened a new pull request, #24606: URL: https://github.com/apache/datafusion/pull/24606
## Which issue does this PR close? None ## Rationale for this change No need to have a separate struct, this open for bugs like: - https://github.com/apache/datafusion/pull/24605 this can just be a trait over the arrow NullBufferBuilder ## What changes are included in this PR? replace MaybeNullBuilder with trait and update usages ## Are these changes tested? existing tests ## Are there any user-facing changes? no, the struct was `pub(crate)` and stayed like that -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
