yyang52 commented on PR #35886: URL: https://github.com/apache/arrow/pull/35886#issuecomment-1605883210
Addressed most comments with most CI checks passed (2 fails seem also happening on other PRs). One left thing is hardcoding `window_bits` in `CodecOptions` may easily be out-of-sync with third-party dependency, as mentioned by @wgtmac. Do you have any suggestions/thoughts on that? Please also have a look at other changes I made and see if any other improvements needed. Thanks! @pitrou @mapleFU @wgtmac -- 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]
