Copilot commented on code in PR #51246:
URL: https://github.com/apache/arrow/pull/51246#discussion_r3961702341
##########
python/pyarrow/io.pxi:
##########
@@ -1101,6 +1101,7 @@ cdef class MemoryMappedFile(NativeFile):
----------
new_size : new size in bytes
"""
+ self._assert_open()
check_status(self.handle.get().Resize(new_size))
Review Comment:
Consider releasing the GIL around the native `Resize` call (e.g. `with
nogil:`) for consistency with other I/O wrappers in this file. Resizing can
involve OS-level work and may otherwise block unrelated Python threads while
the resize completes.
--
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]
