Skip to content

feat: Add BROTLI page decompression support for Parquet reads#195

Open
Anamika1608 wants to merge 4 commits intoDataHaskell:mainfrom
Anamika1608:feat/add-brotli-compression
Open

feat: Add BROTLI page decompression support for Parquet reads#195
Anamika1608 wants to merge 4 commits intoDataHaskell:mainfrom
Anamika1608:feat/add-brotli-compression

Conversation

@Anamika1608
Copy link
Copy Markdown
Contributor

@Anamika1608 Anamika1608 commented Apr 15, 2026

this PR solves the issue - #166

@sharmrj
Copy link
Copy Markdown
Contributor

sharmrj commented Apr 22, 2026

I haven't tested it, but would your brotliLibraryCandidates work if I was using Nix?

In the same vein, we might want to look into using extra-libraries and pkgconfig-depends: https://cabal.readthedocs.io/en/3.4/cabal-package.html#:~:text=pkgconfig%2Ddepends%3A,package%20list. (We should probably do the same for zlib as well). The consumer of our library will then need to set extra-lib-dirs if they have a custom install location or an otherwise idiosyncratic installation.

We should also consider using one of the existing bindings to Brotli available on Hackage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants