dsmiley commented on code in PR #4892: URL: https://github.com/apache/solr/pull/4892#discussion_r4106757257
########## 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: Maybe my choice of "liability" was poor. I'm basically saying that code has a maintenance cost, and documented behavior elevates the cost. Of course we need to document useful/foundational behavior. But we shouldn't document every last bit of behavior/setting in the code. -- 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]
