jorisvandenbossche commented on issue #50398: URL: https://github.com/apache/arrow/issues/50398#issuecomment-5526745029
Ah, thanks for the info! Ideally we should integrate those into our existing benchmarks, but that is a bit complicated now (certainly for Python, where we have a bunch of benchmarks in `/python/benchmarks` but those are essentially unused at the moment and its run configuration has been removed) One more question: you ran https://github.com/apache/arrow/pull/50409 as-is? I.e. with just the code changes, but without enabling building with the limited API? (since that is not yet part of the PR_) Because we might see another / bigger performance impact when actually building with `Py_LIMITED_API` set. -- 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]
