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.

Sign up for GitHub

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

Jump to bottom

Hystrix stream not compatible with turbine #225

Closed
julekgwa opened this issue Sep 6, 2018 · 0 comments · Fixed by #226
Closed

Hystrix stream not compatible with turbine #225

julekgwa opened this issue Sep 6, 2018 · 0 comments · Fixed by #226
Assignees
@helio-frota

Comments

@julekgwa
Copy link
Contributor

julekgwa commented Sep 6, 2018

Node.js Version:
v8.11.0
Operating System:
Darwin julekgwa-MacBook-Pro.local 17.7.0 Darwin Kernel Version 17.7.0
Steps to Produce Error:

The hystrix stream data is not compatible with turbine, the specs says that currentTime is a number but the provided type is of date string in hystrix-formatter.js currentTime: new Date() see https://github.com/Netflix/Hystrix/wiki/Metrics-and-Monitoring#metrics-publisher
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@helio-frota helio-frota

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

changed currentTime to number as specified in the docs julekgwa/opossum
3 participants
@lance @julekgwa @helio-frota