dinse commented on code in PR #46755: URL: https://github.com/apache/arrow/pull/46755#discussion_r2199073574
########## python/pyarrow/includes/libarrow_python_config.pxd: ########## @@ -0,0 +1,29 @@ +# Licensed to the Apache Software Foundation (ASF) under one Review Comment: I am new to Cython and was getting ambiguous overload compilation errors from both GetBuildInfos with a variety of approaches. I did this [dummy namespace pxd](https://cython.readthedocs.io/en/latest/src/userguide/sharing_declarations.html#using-cimport-to-resolve-naming-conflicts) to address naming conflicts; the docs say > "There won’t be any actual [libarrow_python_config] module at run time, but that doesn’t matter; the [libarrow_python_config].pxd file has done its job of providing an additional namespace at compile time." Anyways, I agree it makes sense to keep things in libarrow_python if possible, and I've since found a way to do that. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
