-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Parse 'LogEntry.receiveTimestamp' -> 'received_timestamp' attr. #6143
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
Conversation
Logging system tests flake in |
19955e1
to
fd93f96
Compare
Rebased to trigger a new build. |
/cc @qingling128 |
@tseaver - Oh, great! Glad to see this being added. Thanks for the heads up! 👍 |
c8723aa
to
0610e1f
Compare
Rebased to pick up the various logging systest flakiness fixes landed today. |
0610e1f
to
3e90aa1
Compare
@crwilcox, @theacodes This should be ready for review. |
3e90aa1
to
35d69e6
Compare
It is set only when parsing a server response.
Closing in favor of #6229, which includes these commits, but also adds other missing log entry fields, and refactors |
Closes #5601.