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 plugin #1734

Closed
wants to merge 6 commits into from
Closed

Solr plugin #1734

wants to merge 6 commits into from

Conversation

ljagiello
Copy link
Contributor

Required for all PRs:

  • CHANGELOG.md updated (we recommend not updating this until the PR has been approved by a maintainer)
  • Sign CLA (if not already signed)
  • README.md updated (if adding a new plugin)

Solr plugin collecting metrics from Apache Solr server.

Issue (#278)

servers = ["http://localhost:8983"]

## specify a list of one or more Solr cores
cores = ["main"]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thoughts on having the plugin capture all cores by default without having to manually specify them?

Copy link
Contributor Author

@ljagiello ljagiello Sep 9, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed - baf3246

@phemmer
Copy link
Contributor

phemmer commented Sep 9, 2016

Just a thought, but would it be possible to come up with a generic way of just passing the metrics straight through, instead of having to hard code a struct with specific json tags? It would make the plugin more future proof.

@ljagiello
Copy link
Contributor Author

@phemmer @sparrc any other comments/suggestions?

@ghost
Copy link

ghost commented Oct 27, 2016

@sparrc any chance for a next release?

@sparrc
Copy link
Contributor

sparrc commented Oct 27, 2016

@ljagiello could you provide an example output of the metrics gathered in the README?

@sparrc
Copy link
Contributor

sparrc commented Oct 27, 2016

it could be merged for 1.2

@ljagiello
Copy link
Contributor Author

@sparrc done.

@ljagiello ljagiello closed this Nov 9, 2016
@ljagiello ljagiello mentioned this pull request Nov 9, 2016
3 tasks
@ljagiello
Copy link
Contributor Author

ljagiello commented Nov 9, 2016

mess up with this one during rebase - new PR here: #2019

@phemmer
Copy link
Contributor

phemmer commented Nov 9, 2016

Just FYI, you don't have to open a new PR. As long as the branch name is the same, you can do a force push and PR will get updated.

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.

3 participants