-
Notifications
You must be signed in to change notification settings - Fork 4
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 all messages received #6
Comments
will need to have these fields in the log:
|
@araskachoi regarding hop count, how can I add 1 to the message hop count before sending to the next subscriber? Did you implement that functionality in your original implementation? If so, could you point me to the line in your code, please? Here's the TODO in mine: |
We did not have this implemented inside ours. I found this which might be what we need If this does not give what we need, we can just implement this in our application logic to trace the previous sender and keep tracing it back to the origin, which will be the number of hops. Doing it retroactively like this will be tedious and computationally exhaustive but cannot be helped if all other options do not work. |
We will need to ask PL if we can alter the source code to make sure that we can edit the message struct to increase the hop count. @adam-hanna will ask PL about this and update on what they say! |
I asked this question to Steve Allen yesterday, along with asking for his signoff on our host implementation. I'll revert when he responds. We can always just do this via our analysis layer. I don't think it'll be necessary to edit the PL gossip library. |
Upon receiving a message, the program should increase its number of hops. It should then log to stdout the contents of the message. The log line should be timestamped.
The text was updated successfully, but these errors were encountered: