ForeverAngry commented on PR #2431: URL: https://github.com/apache/iceberg-python/pull/2431#issuecomment-3258196666
Yeah, @kevinjqliu started an issue to trackthisnwork here: https://github.com/apache/iceberg-python/issues/2427 > It looks like there's other cases of this throughout the codebase, such as [`MaintenanceTable`](https://github.com/apache/iceberg-python/blob/52d810efb62e39ec6d8d6a2f4cd2cad8165e2d2c/pyiceberg/table/maintenance.py#L31 ) as well as [locations with both class variables and instance variables of the same name](https://github.com/apache/iceberg-python/blob/52d810efb62e39ec6d8d6a2f4cd2cad8165e2d2c/pyiceberg/utils/parsing.py#L26). It might be worth searching the repo and cleaning this up -- 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