dhruv-pratap commented on PR #6019: URL: https://github.com/apache/iceberg/pull/6019#issuecomment-1284332682
High level summary of changes done as part of the PR: 1. Rest Catalog encapsulates a `requests.Session` 2. Configure `Session` object with SSL config, if provided, during initialization. 3. Configure existing headers one-time in the `session` object during initialization. 4. Use the one-time initialized `Session` object for all further REST calls. 5. Add unit tests for SSL config initiation, and enforce HTTP header checks in existing REST mock tests. -- 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]
