nastra opened a new pull request, #14448: URL: https://github.com/apache/iceberg/pull/14448
The [REST spec](https://github.com/apache/iceberg/blob/6319712b612b724fedbc5bed41942ac3426ffe48/open-api/rest-catalog-open-api.yaml#L225) currently uses `%1F` as the UTF-8 encoded namespace separator for multi-part namespaces. This causes [issues](https://github.com/apache/iceberg/issues/10338), since it's a [control character](https://www.compart.com/en/unicode/category/Cc) and the [Servlet spec](https://jakarta.ee/specifications/servlet/6.0/jakarta-servlet-spec-6.0.html#uri-path-canonicalization) can reject such characters. This PR makes the hard-coded namespace separator configurable by giving servers an option to send an **optional** namespace separator instead of `%1F`. The configuration part is entirely optional for REST server implementers and there's no behavioral change for existing installations. The actual implementation for this can be seen at https://github.com/apache/iceberg/pull/10877 -- 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]
