blackmwk opened a new pull request, #3101: URL: https://github.com/apache/iceberg-rust/pull/3101
## Which issue does this PR close? - Closes #3095. ## What changes are included in this PR? - Replace manual Memory catalog property extraction with the shared `Properties` derive. - Preserve warehouse validation and forward unmodeled properties to downstream consumers. - Add focused coverage for typed and retained catalog properties. ## Are these changes tested? - `cargo test -p iceberg --lib catalog::memory::catalog::tests::test_catalog_properties` - `cargo test -p iceberg --lib catalog::memory::catalog::tests::test_create_table_throws_error_if_table_location_and_namespace_location_and_warehouse_location_are_missing` - `cargo clippy -p iceberg --lib --all-features -- -D warnings` - `cargo fmt --all -- --check` ## AI Disclosure This change was developed with assistance from OpenAI Codex. The contributor reviewed the resulting diff and test output. -- 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]
