NoahKusaba opened a new pull request, #3407: URL: https://github.com/apache/iceberg-rust/pull/3407
## Which issue does this PR close? - No issue. Follow-up to #3286, where @comphead suggested this rename. ## What changes are included in this PR? Renames the loader's private type alias `CatalogBuilderFactory` to `BoxedCatalogBuilderFactory`. The factories in `CATALOG_REGISTRY` return a `Box<dyn BoxedCatalogBuilder>`, not a `CatalogBuilder`, so the alias is now named after the trait it produces. The doc comments on the alias and on `CATALOG_REGISTRY` are updated to match: the old alias doc repeated its own name, and the registry holds builder factories, not builders. The alias is private, so there is no public API change, and `public-api.txt` is unchanged. ## Are these changes tested? No behaviour changes, so no new tests. `cargo fmt --all -- --check`, `cargo clippy -p iceberg-catalog-loader --all-targets --all-features -- -D warnings`, `cargo test -p iceberg-catalog-loader --lib` and `cargo public-api -p iceberg-catalog-loader` (compared against `public-api.txt`) pass locally. ## AI Disclosure - Used Claude Code to make the rename and draft this description. I reviewed the change. -- 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]
