ctargett commented on a change in pull request #371:
URL: https://github.com/apache/solr/pull/371#discussion_r739196929



##########
File path: solr/solr-ref-guide/src/major-changes-in-solr-9.adoc
##########
@@ -181,6 +181,8 @@ The only reason QEC supported the data directory was to 
support loading its chan
 That feature now works from the configset dir too.
 SolrCloud doesn't support that but may sometime.
 
+* If you plan to use a rolling upgrade strategy from an 8.x version prior to 
8.5, you'll first need to do a rolling upgrade to the latest 8.x version. This 
is due to a change in the Overseer response format serialization in version 
8.5. See <<solr-upgrade-notes.adoc#_solr_8_5,8.5 Upgrade Notes>> for details.
+

Review comment:
       Instead of underscores, use hyphens, so `#solr-8-5`. 
   
   Because what you're trying to link to is a header (h3 in this case), the 
link is auto-created during conversion to HTML. The syntax is described here: 
https://github.com/apache/solr/blob/main/dev-docs/ref-guide/asciidoc-syntax.adoc#link-to-a-section-on-another-page.
   
   Since this is an existing section that's been published in several versions 
of the Guide, you could also look at the existing published page to determine 
the auto-created link.




-- 
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: issues-unsubscr...@solr.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



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

Reply via email to