nastra commented on code in PR #14448:
URL: https://github.com/apache/iceberg/pull/14448#discussion_r2481928848
##########
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:
while this would technically be possible with the java client, it would be
more challenging with other client implementations as you'd need to keep track
of each client version where you know that this is a newer client. Hence I
think we should always treat `0x1F` as the legacy separator
--
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]
