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:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
