You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2022/06/16 22:13:59 UTC
[GitHub] [iceberg] Fokko opened a new pull request, #5068: Python: Replace vars with cached_property decorator
Fokko opened a new pull request, #5068:
URL: https://github.com/apache/iceberg/pull/5068
Instead of doing the caching ourselves, functools has a nice decorator that can do that for us. I also removed the type ignore to avoid having an accidental None.
--
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-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
[GitHub] [iceberg] danielcweeks merged pull request #5068: Python: Replace vars with cached_property decorator
Posted by GitBox <gi...@apache.org>.
danielcweeks merged PR #5068:
URL: https://github.com/apache/iceberg/pull/5068
--
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-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
[GitHub] [iceberg] Fokko commented on pull request #5068: Python: Replace vars with cached_property decorator
Posted by GitBox <gi...@apache.org>.
Fokko commented on PR #5068:
URL: https://github.com/apache/iceberg/pull/5068#issuecomment-1158237020
@samredai would you have time to look at this? Since you're the original author 👍🏻
--
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-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org