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

[WIP] Update heartbeat event structure #4054

Closed
wants to merge 3 commits into from

Conversation

ruflin
Copy link
Member

@ruflin ruflin commented Apr 19, 2017

This is a first take on changing the event structure to #3406 (comment)

  • move up to status -> up / down. In the example it is status.type: up? Reason?

This is a first take on changing the event structure to elastic#3406 (comment)

* move up to status -> up / down. In the example it is status.type: up? Reason?
@ruflin ruflin added Heartbeat in progress Pull request is currently in progress. review labels Apr 19, 2017
How do we manage that http could also write into the tcp namespace?
@ruflin
Copy link
Member Author

ruflin commented Apr 24, 2017

Closing in favor of #4091

@ruflin ruflin closed this Apr 24, 2017
@ruflin ruflin deleted the heartbeat-event-structure branch April 24, 2017 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Heartbeat in progress Pull request is currently in progress. review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant