You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucene.apache.org by GitBox <gi...@apache.org> on 2021/02/10 16:40:53 UTC
[GitHub] [lucene-solr] thelabdude opened a new pull request #2346: SOLR-15145: System property to control whether base_url is stored in state.json to enable back-compat with older SolrJ versions
thelabdude opened a new pull request #2346:
URL: https://github.com/apache/lucene-solr/pull/2346
backport of https://issues.apache.org/jira/browse/SOLR-15145 to 8.8 branch
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
[GitHub] [lucene-solr] anshumg commented on a change in pull request #2346: SOLR-15145: System property to control whether base_url is stored in state.json to enable back-compat with older SolrJ versions
Posted by GitBox <gi...@apache.org>.
anshumg commented on a change in pull request #2346:
URL: https://github.com/apache/lucene-solr/pull/2346#discussion_r573892387
##########
File path: solr/CHANGES.txt
##########
@@ -4,6 +4,14 @@ This file lists Solr's raw release notes with details of every change to Solr.
Most people will find the solr-upgrade-notes.adoc file more approachable.
https://github.com/apache/lucene-solr/blob/master/solr/solr-ref-guide/src/solr-upgrade-notes.adoc
+================== 8.8.1 ==================
Review comment:
Does it make sense to add upgrade notes and mention the property that'll allow them to upgrade the server without upgrading the client?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
[GitHub] [lucene-solr] thelabdude commented on a change in pull request #2346: SOLR-15145: System property to control whether base_url is stored in state.json to enable back-compat with older SolrJ versions
Posted by GitBox <gi...@apache.org>.
thelabdude commented on a change in pull request #2346:
URL: https://github.com/apache/lucene-solr/pull/2346#discussion_r573935393
##########
File path: solr/CHANGES.txt
##########
@@ -4,6 +4,14 @@ This file lists Solr's raw release notes with details of every change to Solr.
Most people will find the solr-upgrade-notes.adoc file more approachable.
https://github.com/apache/lucene-solr/blob/master/solr/solr-ref-guide/src/solr-upgrade-notes.adoc
+================== 8.8.1 ==================
Review comment:
Added a note there, let me know if that makes sense?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
[GitHub] [lucene-solr] thelabdude commented on a change in pull request #2346: SOLR-15145: System property to control whether base_url is stored in state.json to enable back-compat with older SolrJ versions
Posted by GitBox <gi...@apache.org>.
thelabdude commented on a change in pull request #2346:
URL: https://github.com/apache/lucene-solr/pull/2346#discussion_r573910079
##########
File path: solr/CHANGES.txt
##########
@@ -4,6 +4,14 @@ This file lists Solr's raw release notes with details of every change to Solr.
Most people will find the solr-upgrade-notes.adoc file more approachable.
https://github.com/apache/lucene-solr/blob/master/solr/solr-ref-guide/src/solr-upgrade-notes.adoc
+================== 8.8.1 ==================
Review comment:
The property already defaults to true, which means the `base_url` will be stored again, so really there's nothing people need to do with the sys prop to get an older SolrJ to work with an 8.8.1 server again. In reality, the sys prop is only there for people that are running the latest SolrJ and want to prepare their clusters for 9.x by explicitly setting to `false`. Doubt anyone will actually do that so the sys prop is probably not all that useful.
I still need to test to make sure an upgrade of server nodes from 8.8.0 to 8.8.1 does restore the `base_url` in ZK.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
[GitHub] [lucene-solr] thelabdude merged pull request #2346: SOLR-15145: System property to control whether base_url is stored in state.json to enable back-compat with older SolrJ versions
Posted by GitBox <gi...@apache.org>.
thelabdude merged pull request #2346:
URL: https://github.com/apache/lucene-solr/pull/2346
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
[GitHub] [lucene-solr] HoustonPutman commented on a change in pull request #2346: SOLR-15145: System property to control whether base_url is stored in state.json to enable back-compat with older SolrJ versions
Posted by GitBox <gi...@apache.org>.
HoustonPutman commented on a change in pull request #2346:
URL: https://github.com/apache/lucene-solr/pull/2346#discussion_r573895918
##########
File path: solr/CHANGES.txt
##########
@@ -4,6 +4,14 @@ This file lists Solr's raw release notes with details of every change to Solr.
Most people will find the solr-upgrade-notes.adoc file more approachable.
https://github.com/apache/lucene-solr/blob/master/solr/solr-ref-guide/src/solr-upgrade-notes.adoc
+================== 8.8.1 ==================
Review comment:
Yes, I very much think so. Similar to the fix for the default autoscaling policy in 8.6.1
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org