Skip to content

Update for usage with solr5#7

Open
rfrail3 wants to merge 1 commit intoaverni:masterfrom
rfrail3:master
Open

Update for usage with solr5#7
rfrail3 wants to merge 1 commit intoaverni:masterfrom
rfrail3:master

Conversation

@rfrail3
Copy link
Copy Markdown

@rfrail3 rfrail3 commented Jan 7, 2016

This script can be used in solr5 without problem. I just remove the v.4 references and test it. Thanks,

@averni
Copy link
Copy Markdown
Owner

averni commented Jan 22, 2016

Thank you for the contribution, I agree about your observation. Originally the plugin was named just solr but I had to change name since it belongs to another plugin in munin contrib and this script is distributed there too.
On the next pull request I can try to propose a refactoring of the solr folder to the contrib mantainers since this is the only plugin in contrib that works for versions >=4 and the plugin named solr works only for versions <=3 (2012).

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