rtpsw commented on PR #34156: URL: https://github.com/apache/arrow/pull/34156#issuecomment-1432702572
It will take a bit of time for me to check, but I suspect static locals would only fix initialization order in C++ but not in Cython. Specifically, I suspect that Cython sees different header definitions for `ConversionOptions` (those generated from the `pxd` file) than C++ does, and doesn't see the inline initialization of its members in the C++ header, leading to an improperly initialized object. I don't mind using static locals to fix the C++ order of initialization, but I'd still suggest adding a constructor to `ConversionOptions`, where its members are initialized, and expose it to Cython. I added a reduced changeset here for just this exposure. -- 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]
