Nishuuzz commented on PR #51753: URL: https://github.com/apache/arrow/pull/51753#issuecomment-6036312721
Both of those are done — filed #52024 for the bug and retitled this accordingly, and the description now follows the template. The remaining question is yours: deriving the list programmatically rather than hardcoding it. I agree that is the better fix, and the obstacle is that the canonical mapping lives in `_ensure_compression()` in `io.pxi` and nothing currently exposes the supported set — neither `Codec` nor the C++ `compression.h` — so it needs a new accessor on the Cython side for `show_info()` to read. I am happy to write that here; the caveat is that I cannot compile the Cython extension in my environment, so I would be leaning on CI to verify it. Otherwise I can leave this as the minimal fix and open a separate issue for the programmatic version. Either is fine by me, whichever you prefer. -- 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]
