roryqi commented on code in PR #10874:
URL: https://github.com/apache/gravitino/pull/10874#discussion_r3161145977
##########
core/src/main/java/org/apache/gravitino/Configs.java:
##########
@@ -500,6 +500,20 @@ private Configs() {}
.checkValue(value -> value > 0,
ConfigConstants.POSITIVE_NUMBER_ERROR_MSG)
.createWithDefault(5 * 60 * 1000L); // Default is 5 minutes
+ public static final ConfigEntry<String> SCHEMA_SEPARATOR =
+ new ConfigBuilder("gravitino.schema.separator")
+ .doc(
+ "The separator used to represent nested namespace hierarchy in
schema names at the "
+ + "API boundary (e.g. ':' for 'A:B:C'). Schema names are
stored internally in "
+ + "EntityStore using '.' as the physical separator. The
configured separator is "
+ + "only used at external API and catalog capability
validation layer. "
+ + "The '.' character is reserved as the internal physical
separator and must "
+ + "not be used as the external separator.")
+ .version(ConfigConstants.VERSION_1_3_0)
+ .stringConf()
+ .checkValue(StringUtils::isNotBlank,
ConfigConstants.NOT_BLANK_ERROR_MSG)
+ .createWithDefault(":");
Review Comment:
Accepted.
--
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]
