rok commented on PR #44070: URL: https://github.com/apache/arrow/pull/44070#issuecomment-2346416340
> > Do we need to do something Python-side? C++ side was covered by #13901. I've added a basic parquet test here and it works for me locally. > > I would be surprised if it works locally out of the box because IIRC the option was disabled by default in C++ for now? (so what we would need on the python side is expose that new `arrow_extensions_enabled` options so that the user can enable it) Although I suppose that you wrote the file with pyarrow, and at that point we give priority to the stored schema, I suppose, even if the option is not enabled. I see. I assumed `arrow_extensions_enabled` was on by default. Do we want to expose a global setter? E.g.: `pyarrow.parquet.set_arrow_extensions_enabled`? Or would it be preferred to add a parameter for `pq.write_table`? > > Question: any idea why pytest is saying `pa.json` is [not callable in CI](https://github.com/apache/arrow/actions/runs/10828781440/job/30044921830?pr=44070#step:6:6727)? Locally this works as expected. > > There might somehow we some conflict with the stdlib module? > [..] > Actually not that, but we have a pyarrow.json module already. So we will have to call this function differently anyway. For > others with a conflict with added a trailing underscore, so could do that here as well. It seems this was the case, changed extension type name to `pa.json_`. -- 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]
