emecii commented on issue #3028: URL: https://github.com/apache/iceberg-python/issues/3028#issuecomment-5613331295
I reviewed the agreed direction in #3083 and the follow-up #3208. Both PRs were closed automatically as stale and there is no open successor, so I prepared a narrow refreshed branch for the first slice: keep the existing `Config()` API, defer configuration-file and environment reads until the first `config`/`get_*` access, and cache the merged result per instance. It does not move `_ENV_CONFIG` or change the catalog-loading API. The branch is https://github.com/emecii/iceberg-python/tree/refactor/lazy-config-loading at `5586c286`. On current `main`, the full unit target passes (4,019 passed, 3 skipped), as do `make lint` and Apache RAT on a clean worktree. @jx2lee @kevinjqliu, is it okay for me to open this narrow PR, or would you prefer the earlier work to be revived/continued instead? OpenAI Codex generated the patch, tests, and this comment and ran the reported checks. No human review is asserted. -- 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]
