thisisnic commented on pull request #10388: URL: https://github.com/apache/arrow/pull/10388#issuecomment-847628316
> Oh, actually one small requested change please: both `Table` and `RecordBatch` inherit the virtual class `ArrowTabular`, so whenever we want to define a method for both `Table` and `RecordBatch` and the implementation of it is exactly the same, we should instead define it on `ArrowTabular`. Could you please change this to use that? The tests can stay the same. Good point, completely missed that, now done! -- 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. For queries about this service, please contact Infrastructure at: [email protected]
