-
Notifications
You must be signed in to change notification settings - Fork 2
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Solr Version #17
Comments
"need to setup a http solr server for testing" The aws instance is still working :) |
It would be nice to add a Solr server to Travis whenever that gets up and running |
Check out: https://github.com/moliware/travis-solr On Sun, Aug 9, 2015 at 7:13 PM, Mike Panciera [email protected]
|
Thanks Sean for the pointer! |
Note to self:
5e6ab1b
These lines might belong below the else block:
SystemInfoHandler sih = new SystemInfoHandler(container);
versionSolr = sih.getVersion();
They might work for both embedded and http solr servers. I'll need to setup a http solr server for testing. I have no idea why there are two constructors, one taking the container and the other doesn't.
http://lucene.apache.org/solr/5_2_1/solr-core/org/apache/solr/handler/admin/SystemInfoHandler.html
(Also, since I might have broken support for http solr servers with that last commit, I need to look into this quickly)
Note to self:
Don't accidentally commit a nullpointerexception in the future...
The text was updated successfully, but these errors were encountered: