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.

Sign up for GitHub

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

Jump to bottom

Timestamps from CommandResult and Event are different #1994

Closed
VasilyStrelyaev opened this issue Aug 12, 2021 · 0 comments · Fixed by #1995 or #2039
Closed

Timestamps from CommandResult and Event are different #1994

VasilyStrelyaev opened this issue Aug 12, 2021 · 0 comments · Fixed by #1995 or #2039
Assignees
@MrCheater
Labels
bug Issues related to defects and incorrect/unexpected feature behavior severity: minor Issues with low impact on the product, e.g., small improvements or imperfections

Comments

@VasilyStrelyaev
Copy link

Timestamps returned in CommandResult and stored in an Event sometimes diverge by one millisecond. This seems to be because here in saveEvent we return an event, instead of the actually stored eventWithCursor (according to @MrCheater)
@MrCheater MrCheater self-assigned this Aug 12, 2021
@MrCheater MrCheater added bug Issues related to defects and incorrect/unexpected feature behavior severity: minor Issues with low impact on the product, e.g., small improvements or imperfections labels Aug 12, 2021
MrCheater added a commit that referenced this issue Aug 12, 2021
@MrCheater
Fix #1994
50f8089
@MrCheater MrCheater mentioned this issue Aug 12, 2021
Merged
MrCheater added a commit that referenced this issue Aug 18, 2021
@MrCheater
Fix #1994 (#1995)
520f8da
@MrCheater MrCheater reopened this Sep 3, 2021
MrCheater added a commit that referenced this issue Sep 10, 2021
@MrCheater
Fix #1994
6e3a462
@MrCheater MrCheater mentioned this issue Sep 13, 2021
Merged
MrCheater added a commit that referenced this issue Sep 13, 2021
@MrCheater
Fix #1994 (#2039)
d3e82ca
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MrCheater MrCheater

Labels
bug Issues related to defects and incorrect/unexpected feature behavior severity: minor Issues with low impact on the product, e.g., small improvements or imperfections
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #1994 reimagined/resolve
Fix #1994 reimagined/resolve
2 participants
@MrCheater @VasilyStrelyaev