flyrain commented on code in PR #3713:
URL: https://github.com/apache/polaris/pull/3713#discussion_r2941892838
##########
polaris-core/src/main/java/org/apache/polaris/core/entity/NamespaceEntity.java:
##########
@@ -33,6 +33,15 @@ public class NamespaceEntity extends PolarisEntity
implements LocationBasedEntit
// RESTUtil-encoded parent namespace.
public static final String PARENT_NAMESPACE_KEY = "parent-namespace";
+ /**
+ * The URL-encoded UTF-8 representation of the namespace separator character
(0x1F).
+ *
+ * <p>This constant mirrors the {@code
+ * org.apache.iceberg.rest.RESTUtil#NAMESPACE_SEPARATOR_URLENCODED_UTF_8}
constant which is not
+ * public.
+ */
+ public static final String NAMESPACE_SEPARATOR_URLENCODED_UTF_8 = "%1F";
Review Comment:
Are we going to support configurable separator in the future?
--
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]
