danielhumanmod opened a new pull request, #383:
URL: https://github.com/apache/polaris/pull/383

   # Description
   
   As the endpoints field was introduced in [Iceberg REST 
spec](https://github.com/apache/iceberg/blob/c0d73f4ef5c16401bdfd62e1745faf2fbbf62177/open-api/rest-catalog-open-api.yaml#L150)
 recently to describe the server capabilities in `/v1/config`, Polaris can 
align with this change by extending the catalog config API to include the 
endpoints field.
   
   Fixes #316 
   
   **Included Changes:**
   - Update `rest-catalog-open-api.yaml` config
   - Add endpoints support in `IcebergCatalogAdapter`
   - Extend `ConfigResponse` and `ResourcePaths` as a temp workaround before 
integrated with Iceberg 1.70
   
   **TODO**
   Since the latest changes related to endpoints are not yet released 
(currently at version 1.6.1) and are planned for release in 1.7.0, we need to 
replace the two extended classes with the Iceberg's implementations after the 
upgrade.
   
   
   ## Type of change
   
   Please delete options that are not relevant.
   
   - [ ] Bug fix (non-breaking change which fixes an issue)
   - [ ] Documentation update
   - [x] New feature (non-breaking change which adds functionality)
   - [ ] Breaking change (fix or feature that would cause existing 
functionality to not work as expected)
   - [ ] This change requires a documentation update
   
   # How Has This Been Tested?
   
   Please describe the tests that you ran to verify your changes. Provide 
instructions so we can reproduce. Please also list any relevant details for 
your test configuration
   
   - [ ] Test A
   - [ ] Test B
   
   **Test Configuration**:
   * Hardware:
   * Toolchain:
   * SDK:
   
   # Checklist:
   
   Please delete options that are not relevant.
   
   - [x] I have performed a self-review of my code
   - [x] I have commented my code, particularly in hard-to-understand areas
   - [x] I have made corresponding changes to the documentation
   - [x] My changes generate no new warnings
   - [x] If adding new functionality, I have discussed my implementation with 
the community using the linked GitHub issue
   


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