singhpk234 commented on code in PR #14448:
URL: https://github.com/apache/iceberg/pull/14448#discussion_r2481690390
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -262,7 +262,9 @@ paths:
An optional namespace, underneath which to list namespaces.
If not provided, all top-level namespaces should be listed.
For backward compatibility, empty string is treated as absent for
now.
- If parent is a multipart namespace, the parts must be separated by
the unit separator (`0x1F`) byte.
+ If parent is a multipart namespace, the parts must be separated by
the namespace separator as
+ indicated via the /config override `namespace-separator`, which
defaults to the unit separator (`0x1F`) byte.
+ To be compatible with older clients, servers must use both the
advertised separator and `0x1F` as valid separators when decoding namespaces.
Review Comment:
Agree, my question was if the server knows the client is new (for example
client send the iceberg sdk version to server as part of header) can they
choose not to treat `0x1F` as seperator ?
--
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]
