merlimat commented on a change in pull request #12663:
URL: https://github.com/apache/pulsar/pull/12663#discussion_r746229975
##########
File path:
pulsar-metadata/src/main/java/org/apache/pulsar/metadata/impl/AbstractMetadataStore.java
##########
@@ -306,4 +315,17 @@ protected static String parent(String path) {
return path.substring(0, idx);
}
+
+ /**
+ * valid path in metadata store should be
+ * 1. not blank
+ * 2. starts with '/'
+ * 3. not ends with '/', except root path "/"
+ */
+ public static boolean isValidPath(String path) {
Review comment:
Maybe we can leave this as package protected
```suggestion
static boolean isValidPath(String path) {
```
--
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]
