You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "jorisvandenbossche (via GitHub)" <gi...@apache.org> on 2023/04/11 12:56:16 UTC

[GitHub] [arrow] jorisvandenbossche commented on pull request #34980: GH-34979: [Python] Create a base class for Table and RecordBatch

jorisvandenbossche commented on PR #34980:
URL: https://github.com/apache/arrow/pull/34980#issuecomment-1503286867

   > The docstring is shared, but ends up being a bit confusing.
   
   On the other hand, a significant part of the usefulness of sharing the implementation is lost if we can't share the docstring? (especially in this example where the function is only a one liner, for others that might be more worth it) 
   What did you find confusing about it? (the need to constantly say "RecordBatch or Table"?)


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