-
Notifications
You must be signed in to change notification settings - Fork 18
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
Log verbosity #7
Comments
Thanks! I'll move this to debug level in the future -- I don't have a collectd version with debug enabled at the moment, so I'm looging at info. :-( |
New version in master that should greatly reduce logging, as well as make things a bit more stable in terms of resource usage. I know my use cases are bit more demanding than most right now. I'll start looking at the redeployed beans issue in JBoss soon... |
Thanks Bryan, I'll try to get an update. I have FastJMX running nicely on my dev env for couple weeks now - I mentioned the workaround I have about Jboss in the other ticket. |
So I updated the fast jmx jar and it does not work anymore. Are there any config changes that I must make as well? I've reverted back to older version. |
Yikes! I cannot think of any required changes... What happens if you include log level configurations?
|
I didn't update anything... I just replaced the jar with the new one. The problem could be that I modified your source - I left a code comment so take a look. The master does not compile with Java 1.5 as defined in the pom. |
I saw that note earlier and have not had a chance to look into it yet.
|
got that Unexpected Throwable: java.lang.NullPointerException error in my fresh setup.
a null pointer exception in the config method was preventing the executor field to be set. |
I've opened issue #14 to track your suggested change. Thanks, rbartl! |
Enhancement request. To have cleaner logs it would be good if after one succesful collection we stop logging success. For example:
[2014-09-25 18:20:47] FastJMX plugin: [failed:0, canceled:0, successful:11] in 8ms with: 2 threads
The text was updated successfully, but these errors were encountered: