rdblue opened a new pull request, #6150:
URL: https://github.com/apache/iceberg/pull/6150

   This PR adds query params to the REST config route that are used to sync 
configuration between the REST client and service.
   
   Right now, this sends the only client catalog property that needs to be in 
sync between the client and service, the warehouse location. Iceberg catalogs 
allow setting the warehouse location as standard catalog config on the client. 
This introduces a problem when using those catalog implementations behind the 
REST API because the service doesn't know what the client's warehouse location 
was set to.
   
   To keep the client config in sync, this sends the properties to the config 
route, where the service can choose how to handle the client config. In the 
tests, the service overrides the client's warehouse config to ensure the two 
are in sync.


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

Reply via email to