-
-
Notifications
You must be signed in to change notification settings - Fork 53
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
collectd_graphite_encoder reads only one measurement #12
Comments
@rashshah thanks for the report. I will have a look asap. |
@rashshah could you give me an example message that is not working for you? This would make debugging a lot easier for me ;-) |
I think I found the bug. Please check in master. |
Changes looks good and works ok. One suggestion, in exception instead of "Error in encoder", say "empty line". Every message from kafka has the empty line in end, and this message will become annoying. Alternatively add check : "if line " Pls release next version. Regards, rashmikant |
New version 0.7.2 is out now. :-) |
Thanks. What is kafka-influxdb tightly coupled with requests=2.8.0? I have some other python packages, which uses 2.8.1, then kafka-influxdb throws error. Traceback (most recent call last): From: Matthias Endler <[email protected]mailto:[email protected]> New version 0.7.2 is out now. :-) Reply to this email directly or view it on GitHubhttps://github.com//issues/12#issuecomment-152464969. |
@rashshah you're right. That shouldn't be the case. What I will do is follow the guideline on http://stackoverflow.com/a/7085000/270334. Will create a ticket for that. Thanks for the hint. |
@rashshah this should be fixed in master now. Would be glad for any feedback. |
collectd_graphite_encoder, when there are multiple measurements in kafka message, it adds only one measurement in influxdb. Seems like problem in for loop. Is it known issue?
The text was updated successfully, but these errors were encountered: