[GitHub] [incubator-pinot] mcvsubbu commented on a change in pull request #4287: Adding dataset metadata and its utility class

[GitBox <gi...@apache.org>]

mcvsubbu commented on a change in pull request #4287: Adding dataset metadata and its utility class
URL: https://github.com/apache/incubator-pinot/pull/4287#discussion_r291635913
 
 

 ##########
 File path: pinot-common/src/main/java/org/apache/pinot/common/metadata/ZKMetadataProvider.java
 ##########
 @@ -57,6 +57,7 @@ private ZKMetadataProvider() {
   private static final String PROPERTYSTORE_INSTANCE_CONFIGS_PREFIX = "/CONFIGS/INSTANCE";
   private static final String PROPERTYSTORE_CLUSTER_CONFIGS_PREFIX = "/CONFIGS/CLUSTER";
   private static final String PROPERTYSTORE_SEGMENT_MERGE_LINEAGE = "/SEGMENT_MERGE_LINEAGE";
+  private static final String PROPERTYSTORE_DATASET_METADATA_PREFIX = "/DATASET_METADATA";
 
 Review comment:
   I am not convinced that we need a separate METADATA folder.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org