epugh commented on code in PR #4892:
URL: https://github.com/apache/solr/pull/4892#discussion_r4019337520


##########
solr/solr-ref-guide/modules/configuration-guide/pages/solr-properties.adoc:
##########
@@ -42,6 +42,10 @@ NOTE: Properties marked with "!" indicate inverted meaning 
between pre Solr 10 a
 
 
|solr.cloud.prep.recovery.read.timeout.additional.ms|prepRecoveryReadTimeoutExtraWait|8000|Specifies
 additional milliseconds to wait during recovery read operations in SolrCloud 
mode.
 
+|solr.cloud.prs.enabled|solr.prs.default|false|Controls the default value for 
Per-Replica State (PRS) when creating a collection that doesn't explicitly 
specify the `PRS` parameter.
+
+|solr.cloud.skip.autorecovery.enabled|solrcloud.skip.autorecovery|false|Controls
 whether automatic recovery of out-of-sync replicas is skipped in SolrCloud 
mode. Intended for testing and diagnostics, not production use.

Review Comment:
   I tend to operate off of patterns...  and the pattern in the properties was 
"make sure they are documented"...   So I documnted them...   But you are 
right, they don't make sense to a Ref Guide user.  I think if they started wtih 
`test.solr.` then it would be clearer their nature...



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to