chairmank commented on pull request #7789: URL: https://github.com/apache/arrow/pull/7789#issuecomment-660120144
> We certainly don't want to do this on the Arrow side (the codecs may be used for something else than Parquet), rather on the Parquet side. What is the right way to check whether the codec is LZ4 so that we can conditionally handle the 8-byte prefix in the Parquet page reader and writer? Comparing the string returned by the `name` method is not ideal. Should we make every `Codec` know its corresponding compression type enum? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
