Skip to content

Conversation

@rahulgoswami
Copy link
Contributor

@rahulgoswami rahulgoswami commented Jan 23, 2026

https://issues.apache.org/jira/browse/SOLR-XXXXX

Description

This is the opposite of what happens on branch_10x (#4074) since the default mode on 9x is user-managed.

Tests

"solr start -e techproducts --cloud" ==> now starts in cloud mode
"solr start -e techproducts" ==> starts in standalone mode

Checklist

Please review the following and check all that apply:

  • I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
  • I have created a Jira issue and added the issue ID to my pull request title.
  • I have given Solr maintainers access to contribute to my PR branch. (optional but recommended, not available for branches on forks living under an organisation)
  • I have developed this patch against the main branch.
  • I have run ./gradlew check.
  • I have added tests for my changes.
  • I have added documentation for the Reference Guide
  • I have added a changelog entry for my change

@rahulgoswami rahulgoswami changed the title --cloud option is not honored when running example (-e) --cloud option is not honored when running example (-e) on Windows Jan 23, 2026
@epugh
Copy link
Contributor

epugh commented Jan 23, 2026

I am adding a changelog as this looks like a bug that has been aroudn and should be documented that it's fixed! Thanks!

@epugh
Copy link
Contributor

epugh commented Jan 23, 2026

I don't know why the precommit error is happening.. I checked and the line affecting it for floating point precission was literally fixed two weeks ago by @janhoy. Copilot's suggested fix is what Jan did as well. So, I am going to merge this.

@epugh epugh merged commit 65e8d28 into apache:branch_9x Jan 23, 2026
5 of 7 checks passed
@rahulgoswami
Copy link
Contributor Author

I am adding a changelog as this looks like a bug that has been aroudn and should be documented that it's fixed! Thanks!

Thanks for adding the changelog Eric.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants