Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork1.8k
Passstart command required in Solr 10#10174
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Passstart command required in Solr 10#10174
Uh oh!
There was an error while loading.Please reload this page.
Conversation
eddumelendez commentedApr 7, 2025
Hi@epugh, thanks for your contribution. Do you mind adding a test with version 10.x? |
epugh commentedApr 7, 2025
10x hasn't yet been released (sometime in May to June-ish timeframe) so I don't think I can add the test. Having said that, I'll definitly provide a test when we have a release available. The addition of the word "start" is compatible with 8 and 9, and will be required in 10. |
start command required in Solr 1080ae5d7 intotestcontainers:mainUh oh!
There was an error while loading.Please reload this page.
eddumelendez commentedApr 7, 2025
Thanks again for your contribution,@epugh! Glad to see those changes are backward compatible and will make the module ready to support the upcoming Solr version. |
epugh commentedApr 8, 2025
My pleasure! I apprecaite your comments, and it made it a pleasant experience to contribute ;-) |
In 10x we removed the short cut that "bin/solr" was a call to "bin/solr start", now we require you to specify what subcommand you want. This change is backwards compatible with 8x and 9x, and forwards compatible with the upcoming Solr 10 release.