-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add retry in templates to wait realtime notifications #161
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a single sleep is not enough, we need a retry mechanism
time.Sleep(4 * time.Second) | ||
os.Exit(0) | ||
}() | ||
|
||
b := make([]byte, 4096) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should make sure that the buffer has been completely emptied and printed before stopping the snippet: 4k may not be large enough
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I pick this value that should be enough because in the snippets we always print a string with a fixed length
cb9253a
to
7cd127d
Compare
What does this PR do?
Add retry in templates to wait realtime notifications