raulcd commented on PR #49116: URL: https://github.com/apache/arrow/pull/49116#issuecomment-3834960752
We seem to be testing wheel-manylinux free-threaded with cython nightlies, in this case: > https://pypi.anaconda.org/scientific-python-nightly-wheels/simple/cython/3.3.0a0/cython-3.3.0a0-py3-none-any.whl I was taking a look on latest commits from Cython and the failures seem related to the string comparison optimizations on the following commit but I am unsure what the problem is and whether is a regression on Cython or we should fix the comparison to require the gil: https://github.com/cython/cython/commit/46e961eeed1c4c3206dce21377edccfbb57577eb Current CI error: ```python [14/115] Compiling Cython CXX source for lib... FAILED: CMakeFiles/lib_pyx /arrow/python/build/temp.linux-x86_64-cpython-313t/CMakeFiles/lib_pyx cd /arrow/python/build/temp.linux-x86_64-cpython-313t && /opt/python/cp313-cp313t/bin/python -m cython --cplus --warning-errors --no-c-in-traceback -Xfreethreading_compatible=True --directive embedsignature=True --working /arrow/python --output-file /arrow/python/build/temp.linux-x86_64-cpython-313t/lib.cpp /arrow/python/pyarrow/lib.pyx Error compiling Cython file: ------------------------------------------------------------ ... options.field_merge_options = _parse_field_merge_options( "default" if promote_options == "none" else promote_options ) with nogil: options.unify_schemas = promote_options != "none" ^ ------------------------------------------------------------ pyarrow/table.pxi:6290:48: Coercion from Python not allowed without the GIL Error compiling Cython file: ------------------------------------------------------------ ... options.field_merge_options = _parse_field_merge_options( "default" if promote_options == "none" else promote_options ) with nogil: options.unify_schemas = promote_options != "none" ^ ------------------------------------------------------------ pyarrow/table.pxi:6290:48: Operation not allowed without gil ``` This is unrelated to the current fix on the PR but something we have to fix. @pitrou do you know what the problem is / solution could be? -- 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]
