ianmcook commented on pull request #10388: URL: https://github.com/apache/arrow/pull/10388#issuecomment-847349776
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. -- 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]
