jhorstmann opened a new pull request, #11481: URL: https://github.com/apache/arrow-rs/pull/11481
# Which issue does this PR close? <!-- It is not necessary to raise an issue, but it is encouraged if it is a non-trivial enhancement/bug. If this PR relates to an existing issue, but does not fully resolve it, please change the wording below from "Closes #NNN" to "Relates to #NNN" or "Part of #NNN" to avoid false closing an issue. --> - Closes #11480 # Rationale for this change I would like to reuse some of the lower-level parquet implementation outside of this crate, and also benchmark against different Decoder implementations. All other encoders and decoders were already public, so I would assume this was an oversight for BSS and ALP when they were added. <!-- Why are you proposing this change? If this is already explained clearly in the issue then this section is not needed. Explaining clearly why changes are proposed helps reviewers understand your changes and offer better suggestions for fixes. Please try to explain the problem you are trying to solve in terms of the user-visible behavior, rather than the implementation. --> # What changes are included in this PR? Publicly exporting these encoders and decoders and making their constructors and other required traits public. <!-- There is no need to duplicate the description in the issue here but it is sometimes worth providing a summary of the individual changes in this PR. --> # Are these changes tested? Verified locally that they are publicly accessible without warnings about visibility. # Are there any significant user-facing changes? No -- 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]
