dssysolyatin commented on PR #5099: URL: https://github.com/apache/calcite/pull/5099#issuecomment-4972557946
This commit also originated as a fix for incorrect struct field access, since it sets the lambda context before any validation. As a result, parameterTypes holds valid types before we start using it. But anyway, I agree that the scope of this commit is much bigger than the task was meant to be. And as you mentioned, I have already missed part of the work that was done which I need to check. PR itself looks good LGTM, except question about array -- 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]
