Skip to content

Use Stopwatch in PingCommand for more precise timing#827

Merged
mtmk merged 1 commit into
mainfrom
fix-ping-timer
Apr 14, 2025
Merged

Use Stopwatch in PingCommand for more precise timing#827
mtmk merged 1 commit into
mainfrom
fix-ping-timer

Conversation

@mtmk

@mtmk mtmk commented Apr 13, 2025

Copy link
Copy Markdown
Member

It's important to use monotonic timer here for correctness.

It's important to use monotonic timer here for correctness.
@mtmk
mtmk requested a review from to11mtm April 13, 2025 23:59

@caleblloyd caleblloyd left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mtmk
mtmk merged commit 051deb5 into main Apr 14, 2025
@mtmk
mtmk deleted the fix-ping-timer branch April 14, 2025 02:42
mtmk added a commit that referenced this pull request Apr 16, 2025
* Stopped testing with net6.0 (#821)
* Update JetStream models (#832)
* Add direct request-reply (#829)
* Socket Connection Factory (#828)
* Fix proxy with TLS scenario with SocketConnectionFactory (#826)
* Make NatsJSConstants public (#825)
* Use Stopwatch in PingCommand for more precise timing (#827)
* Fix netstandard bugs and tidy up tests (#817)
@mtmk mtmk mentioned this pull request Apr 16, 2025
mtmk added a commit that referenced this pull request Apr 16, 2025
* Stopped testing with net6.0 (#821)
* Update JetStream models (#832)
* Add direct request-reply (#829)
* Socket Connection Factory (#828)
* Fix proxy with TLS scenario with SocketConnectionFactory (#826)
* Make NatsJSConstants public (#825)
* Use Stopwatch in PingCommand for more precise timing (#827)
* Fix netstandard bugs and tidy up tests (#817)
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