nastra commented on PR #14984: URL: https://github.com/apache/iceberg/pull/14984#issuecomment-5180095076
> @nastra the view changes are the largest substantive part of this PR. Since you own the core view implementation, do you want to take a look at the view-related changes too? @manuzhang Is there an overview of all the required view-related changes in the Iceberg codebase? I see the changes in https://github.com/apache/iceberg/pull/14984/changes/fd0d03b72b263471935f9acd45ed9bafb5f052b3 but it's not clear to me why certain changes were necessary. Having additional comments on the PR would help reviewers understand the reasoning/motivation behind changes -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
