JeroenSchmidt commented on PR #3649: URL: https://github.com/apache/iceberg-python/pull/3649#issuecomment-5050598840
Hi @rambleraptor, I think I found a working solution that looks correct. What do you think? See https://github.com/apache/iceberg-python/pull/3649/commits/57ebb876fc6e82399c45341bdbb006cc9b9b138f I also added an [explicit integration test](https://github.com/apache/iceberg-python/pull/3649/commits/29a4f762e0b828263c9d30d52784d471b2b8747f) to validate everything works as expected. Let me know if this solution looks ok and Ill proceed to implement some explicit tests for the `_effective_refs`. -- 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]
