nogil not necessary for function that just accesses Python objects

anjakefala · Sep 22, 2023 · 19a7423 · 19a7423
1 parent bda359b
commit 19a7423
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/python/pyarrow/_parquet_encryption.pxd b/python/pyarrow/_parquet_encryption.pxd
@@ -31,7 +31,7 @@ from pyarrow.lib cimport _Weakrefable
 cdef class CryptoFactory(_Weakrefable):
     cdef shared_ptr[CPyCryptoFactory] factory
     cdef init(self, callable_client_factory)
-    cdef inline shared_ptr[CPyCryptoFactory] unwrap(self) nogil
+    cdef inline shared_ptr[CPyCryptoFactory] unwrap(self)
 
 cdef class EncryptionConfiguration(_Weakrefable):
     cdef shared_ptr[CEncryptionConfiguration] configuration

diff --git a/python/pyarrow/_parquet_encryption.pyx b/python/pyarrow/_parquet_encryption.pyx
@@ -455,7 +455,7 @@ cdef class CryptoFactory(_Weakrefable):
     def remove_cache_entries_for_all_tokens(self):
         self.factory.get().RemoveCacheEntriesForAllTokens()
 
-    cdef inline shared_ptr[CPyCryptoFactory] unwrap(self) nogil:
+    cdef inline shared_ptr[CPyCryptoFactory] unwrap(self):
         return self.factory