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

[DONOT MERGE] Initial heartbeat implementation from Dhanaprakaash #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mbharatheesha
Copy link
Collaborator

@srikrishna3118 @ashish-joglekar This is a very basic pub sub based heartbeat implementation from Dhanaprakaash. I have merely copied the source code from him and put into version tracking without any contribution to the code from myself because Dhanaprakaash doesn't have access to this repo.

So, could one of you please add Dhanaprakaash to this repo? We can also review this code and add comments on this PR.

@mbharatheesha
Copy link
Collaborator Author

@dhanaprakaash Do you want to make any additional changes to this PR before review or is this the implementation for review?

@dhanaprakaash
Copy link

@mbharatheesha sir, I have implemented the basic functionality of the heartbeat which is working fine. Further, I would like to add some functionalities on network statistics such as packet loss, average delay, minimum delay, and so on in order, it makes network analytics tools.

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