andersonm-ibm commented on pull request #10450: URL: https://github.com/apache/arrow/pull/10450#issuecomment-933848434
> > > Thanks for adding the user guide docs! > > A question related to building pyarrow: by adding it all together in _parquet.pyx, this means that if you build pyarrow (`PYARROW_WITH_PARQUET=1`), you actually _need_ to enable the parquet encryption on the C++ side, otherwise you will get runtime errors importing pyarrow.parquet. We need to decide if we are fine with basically requiring the encryption to be enabled for pyarrow, but if we do, it should certainly be documented in the contributing guides on building pyarrow. @jorisvandenbossche , you're right, we mandate enabling parquet encryption for importing pyarrow.parquet. I've added it to documentation and to build scripts. -- 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]
