HoustonPutman commented on code in PR #636: URL: https://github.com/apache/solr-operator/pull/636#discussion_r1346172394
########## docs/upgrade-notes.md: ########## @@ -130,6 +130,8 @@ _Note that the Helm chart version does not contain a `v` prefix, which the downl - The `POD_HOSTNAME` envVar in SolrCloud Pods has been deprecated. Use `POD_NAME` instead. +- Use of the `hostPort` system property placeholder in custom solr.xml files has been deprecated. Use `solr.port.advertise` instead. Review Comment: ```suggestion - Use of the `hostPort` system property placeholder in custom solr.xml files has been deprecated. Use `<int name="hostPort">${solr.port.advertise:80}</int>`, the default value used by Solr, instead. ``` -- 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