benibus commented on PR #14753: URL: https://github.com/apache/arrow/pull/14753#issuecomment-1375962480
@pitrou After looking deeper, what I said earlier isn't entirely correctly. Nested fields are only accepted in a few cases where `FindSortKeys` isn't called. I tightened those up in the latest commit. There were also some inconsistencies in error reporting for missing/invalid `FieldRef`s (particularly in the select-k module), so I attempted to fix those as well. Also, due to the error message changes, one of the python tests is failing in the appveyor build. I figured I'd wait for your opinion before fixing it - but I suspect you'd agree that the new error diagnostics would be more informative. -- 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]
