janhoy commented on code in PR #4236: URL: https://github.com/apache/solr/pull/4236#discussion_r3088071294
########## solr/solr-ref-guide/modules/deployment-guide/pages/solrcloud-distributed-requests.adoc: ########## Review Comment: I felt the arbitrary max 1000 distributed requests limit has been under-communicated and comes as a surprise to people, especially the largest clusters with many shards, who have had to fork solr to change this value. So I wanted to tie this new parameter into the guide in plain english somewhere, not only in the list of sysprops. I appreciate that this is a very corner case toggle and 1000 is sufficient for 95% of users, so perhaps this is the wrong location for such a paragraph. Maybe adding a paragraph to "Major changes in solr 9.1" introducing the utility of the new sysprop, is a better location? Or do we have some Q&A part of the ref guide? -- 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]
