TGooch44 commented on a change in pull request #2866:
URL: https://github.com/apache/iceberg/pull/2866#discussion_r680885590
##########
File path: python/iceberg/api/schema.py
##########
@@ -80,6 +82,15 @@ def lazy_lowercase_name_to_id(self):
return self._lowercase_name_to_id
+ def lazy_id_to_name(self):
+ from .types import index_by_name
Review comment:
@rymurr Great idea, I think this actually links to your comment in
@jun-he 's other PR. There is a fair amount of non-idiomatic python code in
this repo that should be refactored and would eliminate most if not all if
these types of issues. Starting with the easy ones like get rid of all classes
that contain only static methods would be a good place to start, and having an
issue to track against would be helpful.
--
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]
