Skip to content

Conversation

remh
Copy link

@remh remh commented Jan 11, 2013

  1. When the connection fails 3 times in a row, we won't try to open it up again.
  2. The jmx process can be launched with a different priority (not by default)
  3. I added an option in the jmx terminal to do a prefiltering of the values that are fetched which seems to add more stability to the jmx connector
  4. Better failure handling
  5. The check will only dump jmx domains that are actually used

Copy link
Contributor

Choose a reason for hiding this comment

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

if priority == DEFAULT_PRIORITY:

Copy link
Author

Choose a reason for hiding this comment

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

No, if the default_priority is set to another value we will have to use the nice command.
The only case where we won't use nice is when the priority==0

Copy link
Contributor

Choose a reason for hiding this comment

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

Ok, I'll make it a bit more explicit.

@alq666
Copy link
Contributor

alq666 commented Jan 11, 2013

Why create a closure here?

def in_domains(domain):

If the function ends up being passed around, won't domains and approx leak?

@remh
Copy link
Author

remh commented Jan 11, 2013

It's a workaround for solr as solr domain name is not completely known, so for solr we fetch everything and then we filter.
We could ask the user to set the domain name in the config file, but the less things you have to configure the better it is I think.

@remh remh merged commit 8882b09 into master Jan 24, 2013
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