Skip to content

Fix: system servlet for Solr 4.4 and above#6

Closed
luxifer wants to merge 2 commits intoreprovinci:masterfrom
wanadev:solr-4-4-above
Closed

Fix: system servlet for Solr 4.4 and above#6
luxifer wants to merge 2 commits intoreprovinci:masterfrom
wanadev:solr-4-4-above

Conversation

@luxifer
Copy link

@luxifer luxifer commented Jan 2, 2014

No description provided.

@rjkip
Copy link
Contributor

rjkip commented Jan 8, 2014

I don't use the system info. Why was this changed in 4.4+? Does admin/system still exist? How does this affect <4.4 users?

@luxifer
Copy link
Author

luxifer commented Jan 9, 2014

The admin/system does not exist anymore, it's a BC break since solr 4.4. So i suggest to make a tag in this repo and write on the readme it's for solr <4.4 if you want to merge this PR. In solr 4.4 and above the admin/system is replaced by admin/info/system, but it's exactly the same json.

@rjkip
Copy link
Contributor

rjkip commented Jan 13, 2014

Sorry for the slow response. What about a configurable info URL so I don't have to do something radical like a new major release?

@luxifer
Copy link
Author

luxifer commented Jan 14, 2014

It could be a great alternative, i'll make the changes in this PR

@luxifer
Copy link
Author

luxifer commented Jun 30, 2014

Done in #10

@luxifer luxifer closed this Jun 30, 2014
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

Successfully merging this pull request may close these issues.

2 participants