smaheshwar-pltr commented on PR #2178: URL: https://github.com/apache/iceberg-python/pull/2178#issuecomment-3057348447
> Maybe its fine to duplicate the update method in the subclasses and leave a comment about why its there for the cached properties? Yeah I think maybe we should just make a larger change here and having subclasses implementing updates sounds good. Will need to think about how to go about that though I realised that the current PR (that removes only cached properties) doesn't guard against the case where a subclass has a private member. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
