liyafan82 commented on pull request #9964: URL: https://github.com/apache/arrow/pull/9964#issuecomment-817430701
> Looks good! > > IMHO, does this PR change the signature of API a bit? Hopefully, the change does not affect existing code practically. @kiszk Thanks for your feedback. Hopefully, it should not affect existing code, as the new return type `List/Map` is a sub-type of the original return type `Object`. If we change the return type in the other direction, client code can be affected. -- 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: us...@infra.apache.org