mbwaheed commented on a change in pull request #1317: SOLR-13101: Create
metadataSuffix znode only at common shard creating api calls
URL: https://github.com/apache/lucene-solr/pull/1317#discussion_r391124376
##########
File path:
solr/core/src/java/org/apache/solr/store/shared/metadata/SharedShardMetadataController.java
##########
@@ -53,12 +53,12 @@ public SharedShardMetadataController(SolrCloudManager
cloudManager) {
/**
* Creates a new metadata node if it doesn't exist for shared shared index
whose correctness metadata
- * is managed by ZooKeeper
+ * is managed by ZooKeeper. This node should only be created during Shard
creation or Recovery events
*
* @param collectionName name of the collection that needs a metadata node
* @param shardName name of the shard that needs a metadata node
*/
- public void ensureMetadataNodeExists(String collectionName, String
shardName) throws IOException {
+ public void initiateMetadataNode(String collectionName, String shardName)
throws IOException {
Review comment:
Wouldn't it be more appropriate to call it createMetadataNode and fail in
case the node already exists?
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org
