singhpk234 commented on code in PR #16394:
URL: https://github.com/apache/iceberg/pull/16394#discussion_r3271422673
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -311,6 +313,7 @@ paths:
summary: Create a namespace
parameters:
- $ref: '#/components/parameters/idempotency-key'
+ - $ref: '#/components/parameters/client-capabilities'
Review Comment:
Thats fair, the thought process i had was its eq to x-iceberg-client version
header from SDK, we just broadcast it for all the call and the server can
choose how to react to it, agree namespace is not tied some capability we are
broadcasting so it might be redundant here, though this would mean each
endpoint would have to make a decision if they need capability header, which we
can add as we go
--
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]
