AlenkaF commented on PR #39983: URL: https://github.com/apache/arrow/pull/39983#issuecomment-3557568518
Thank you for a detailed explanation in the issue: https://github.com/apache/arrow/issues/33473#issuecomment-1932101944. I think the problem and the solution are clear to me. The main thing is that the `if isinstance(descriptor, dict):` check is now removed. I have run the test on current main to check it does fail. Could you rebase on main @eirki in order to resolve conflicts and run the CI? Then I will ask another maintainer for a review after which this will be ready to get merged. Thank you for your patience on getting the PR reviewed! -- 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]
