You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "hendrikmakait (via GitHub)" <gi...@apache.org> on 2023/12/06 08:53:54 UTC
[I] [Python] `.nbytes` blocks the GIL while calculating the data size [arrow]
hendrikmakait opened a new issue, #39096:
URL: https://github.com/apache/arrow/issues/39096
### Describe the enhancement requested
The `.nbytes` holds the GIL while computing the data size in C++, which has caused performance issues in Dask because threads were blocking each other. Am I missing a valid reason for blocking the GIL during the computation? From my understanding, there should be no possible race conditions. Similarly, I noticed that `.get_total_buffer_size` holds the GIL as well, but I'm not sure if that's a problem because the computation is much cheaper.
### Component(s)
Python
--
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: issues-unsubscribe@arrow.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Python] `.nbytes` should release the GIL while calculating the data size [arrow]
Posted by "jorisvandenbossche (via GitHub)" <gi...@apache.org>.
jorisvandenbossche commented on issue #39096:
URL: https://github.com/apache/arrow/issues/39096#issuecomment-1842802119
No, I don't think there is a specific reason for not releasing the GIL, apart from missing to do that when this was added (or assuming this was cheap, I think initially `nbytes` returned the cheap number)
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Python] `.nbytes` should release the GIL while calculating the data size [arrow]
Posted by "jorisvandenbossche (via GitHub)" <gi...@apache.org>.
jorisvandenbossche closed issue #39096: [Python] `.nbytes` should release the GIL while calculating the data size
URL: https://github.com/apache/arrow/issues/39096
--
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: issues-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org