Re: [PR] Core: Add missing REST endpoint definitions [iceberg]

["ebyhr (via GitHub)" <gi...@apache.org>]

ebyhr commented on PR #11756:
URL: https://github.com/apache/iceberg/pull/11756#issuecomment-2614600635

   This PR seems troublesome because it prevents other query engines from upgrading to 1.8.0 if they have tests with older versions. For instance, Trino can't upgrade to this version because Polaris uses 1.7.1 & Unity uses 1.5.2 and they don't contain these endpoints in `/v1/config` response yet. We have to disable tests using those containers or wait until they fix the implementation. 
   
   Also, can we add this PR to [1.8.0 milestone](https://github.com/apache/iceberg/milestone/50?closed=1)? It requires actions by REST catalog developers. 
   
   cc: @nastra @Fokko @amogh-jahagirdar


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org