Skip to content
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

Open
nabihlme opened this issue Aug 8, 2015 · 4 comments
Open

Solr Version #17

nabihlme opened this issue Aug 8, 2015 · 4 comments

Comments

@nabihlme
Copy link
Collaborator

nabihlme commented Aug 8, 2015

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...

@lianyi
Copy link
Collaborator

lianyi commented Aug 9, 2015

"need to setup a http solr server for testing"

The aws instance is still working :)

@averagehat
Copy link
Collaborator

It would be nice to add a Solr server to Travis whenever that gets up and running

@seandavi
Copy link

Check out:

https://github.com/moliware/travis-solr

On Sun, Aug 9, 2015 at 7:13 PM, Mike Panciera [email protected]
wrote:

It would be nice to add a Solr server to Travis whenever that gets up and
running


Reply to this email directly or view it on GitHub
#17 (comment).

@lianyi
Copy link
Collaborator

lianyi commented Aug 22, 2015

Thanks Sean for the pointer!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants