[ 
https://issues.apache.org/jira/browse/SOLR-13169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17134939#comment-17134939
 ] 

Gus Heck commented on SOLR-13169:
---------------------------------

Corrections from another read through, and documentation for other parameters. 
Choosing not to document `waitForFinalState` at this time because it's unclear 
what value it has. This command already has a wait for the completion of the 
add command and causing the add command to wait/block on it's own doesn't seem 
useful (alternately, my understanding of that parameter is flawed and I 
shouldn't write it into the docs). Opened SOLR-14568 which may change the docs 
for timeout slightly. This turned into a lot more than originally anticipated, 
so attaching a patch summarizing changes to the ref guide in case that helps 
folks look over what I've done.  Given no objections I'll port whatever applies 
to 8.x down to 8.x next weekend (and fix any objections). 

> Move Replica Docs need improvement (V1 and V2 introspect)
> ---------------------------------------------------------
>
>                 Key: SOLR-13169
>                 URL: https://issues.apache.org/jira/browse/SOLR-13169
>             Project: Solr
>          Issue Type: Improvement
>          Components: v2 API
>            Reporter: Gus Heck
>            Priority: Major
>         Attachments: SOLR-13169.patch, screenshot-1.png, testing.txt
>
>
> At a minimum required parameters should be noted equally in both places. 
> Conversation with [~ab] indicates that there are also some discrepancies in 
> what is and is not actually required in docs vs code. ("in MoveReplicaCmd if 
> you specify “replica” then “shard” is completely ignored")
> Also in v2 it seems shard might be inferred from the URL and in that case 
> it's not clear if the URL or the json takes precedence.
> From introspect:
> {code:java}
>                 "move-replica": {
>                     "type": "object",
>                     "documentation": 
> "https://lucene.apache.org/solr/guide/collections-api.html#movereplica";,
>                     "description": "This command moves a replica from one 
> node to a new node. In case of shared filesystems the `dataDir` and `ulogDir` 
> may be reused.",
>                     "properties": {
>                         "replica": {
>                             "type": "string",
>                             "description": "The name of the replica"
>                         },
>                         "shard": {
>                             "type": "string",
>                             "description": "The name of the shard"
>                         },
>                         "sourceNode": {
>                             "type": "string",
>                             "description": "The name of the node that 
> contains the replica."
>                         },
>                         "targetNode": {
>                             "type": "string",
>                             "description": "The name of the destination node. 
> This parameter is required."
>                         },
>                         "waitForFinalState": {
>                             "type": "boolean",
>                             "default": "false",
>                             "description": "Wait for the moved replica to 
> become active."
>                         },
>                         "timeout": {
>                             "type": "integer",
>                             "default": 600,
>                             "description": "Timeout to wait for replica to 
> become active. For very large replicas this may need to be increased."
>                         },
>                         "inPlaceMove": {
>                             "type": "boolean",
>                             "default": "true",
>                             "description": "For replicas that use shared 
> filesystems allow 'in-place' move that reuses shared data."
>                         }
> {code}
> From ref guide for V1:
> MOVEREPLICA Parameters
> collection
> The name of the collection. This parameter is required.
> shard
> The name of the shard that the replica belongs to. This parameter is required.
> replica
> The name of the replica. This parameter is required.
> sourceNode
> The name of the node that contains the replica. This parameter is required.
> targetNode
> The name of the destination node. This parameter is required.
> async
> Request ID to track this action which will be processed asynchronously.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to