danepitkin commented on code in PR #43729:
URL: https://github.com/apache/arrow/pull/43729#discussion_r1725157645


##########
python/pyarrow/lib.pxd:
##########
@@ -519,6 +519,7 @@ cdef class Table(_Tabular):
         CTable* table
 
     cdef void init(self, const shared_ptr[CTable]& table)
+    cdef void _assert_cpu(self) except *

Review Comment:
   I thought so too, but I get a runtime error if I leave it out. Since its a 
`cdef` function, cython adds it to a vtable for that class. I guess Cython 
needs the parent class to have an explicitly defined vtable entry?
   
   ```
   >   self._assert_cpu()
   E   AttributeError: 'pyarrow.lib.RecordBatch' object has no attribute 
'_assert_cpu'
   ```



-- 
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]

Reply via email to