pitrou commented on PR #49334: URL: https://github.com/apache/arrow/pull/49334#issuecomment-3925564723
> The C++ writer still rejects bloom filters when file encryption is enabled (ParquetException::NYI in file_writer.cc). Because of that, the tests here build an encrypted payload in memory to exercise the reader path. How did you generate the encrypted payload? Ideally we should add a test file in https://github.com/apache/parquet-testing/tree/master/data (perhaps generated with another Parquet implementation?) > Do you think we should add writer-side support for encrypted bloom filters in this PR as well, or handle that in a follow-up? It depends if you feel comfortable doing it! -- 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]
