epugh commented on code in PR #4892: URL: https://github.com/apache/solr/pull/4892#discussion_r4106403215
########## 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: ? Not sure what you mean? It's all there in the source? I don't see what you mean by liability? There are lots of knobs that if you mess them up bad things happen! Now, if a knob or dial isn't meant to be used in production, like these testing ones, which we documented are for testing, shouldn't be in the docs because it's not sueful to a user, that part I grok and agree with. -- 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]
