danepitkin commented on PR #34980:
URL: https://github.com/apache/arrow/pull/34980#issuecomment-1500597881
The first commit moves the `take()` method from the individual `Table` and
`RecordBatch` classes to the base class `_Table`. The docstring is shared, but
ends up being a bit confusing. I think the best path forward is to refactor
this and create an internal method on the `_Table` class e.g. `_Table._take()`
and override it in the subclasses with a subclass-specific docstring.
E.g.
```
cdef class Table(_Table):
def take(self, object indices):
""" <Class-specific docstring> """
return self._take(indices)
--
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]
