Fokko opened a new pull request, #8545: URL: https://github.com/apache/iceberg/pull/8545
A follow-up on https://github.com/apache/iceberg/pull/8521 with a test this time. This also simplifies the types by removing `ReadableDecoder`. This was needed since it also pulls in the Cython implementation: ```python ReadableDecoder = Union[BinaryDecoder, CythonBinaryDecoder] ``` And would then also break the code. The `new_decoder` makes the testing a bit easier, and I also don't like too much logic when importing the packages. -- 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]
