shyjsarah opened a new pull request, #570:
URL: https://github.com/apache/paimon-rust/pull/570

   ### Purpose
   
   Linked issue: close #569
   
   REST catalog servers can return both configuration defaults and mandatory 
overrides. The Rust client currently ignores `overrides`, so client options can 
incorrectly disable server-required settings.
   
   ### Brief change log
   
   - Add `overrides` to `ConfigResponse`, defaulting to an empty map for 
backward compatibility.
   - Apply the merge order `defaults < client options < overrides`.
   - Include overrides when converting a config response directly to `Options`.
   - Add regression coverage for precedence, old responses without overrides, 
and direct conversion.
   
   ### Tests
   
   - `cargo fmt --all -- --check`
   - `cargo test -p paimon --lib`
   - `cargo test -p paimon config_response_ --lib`
   - `cargo clippy -p paimon --lib --tests -- -D warnings`
   
   ### API and Format
   
   Adds the REST API `overrides` field to `ConfigResponse`. No storage format 
changes.
   
   ### Documentation
   
   No user-facing documentation changes are required.


-- 
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]

Reply via email to