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.

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

perf: Avoid many System.nanoTime calls in sharding passivation strategies #31834

Merged
merged 2 commits into from
Feb 16, 2023

Conversation

patriknw
Copy link
Member

  • The recency based passivation strategies (new default) is calling System.nanoTime for each message.
  • Add a ScheduledClock that updates the time once per second, but still increments time for subsequent calls

@patriknw patriknw force-pushed the wip-scheduled-clock-patriknw branch from 2915526 to c68897e Compare February 10, 2023 10:53
…gies

* The recency based passivation strategies (new default) is calling System.nanoTime
  for each message.
* Add a ScheduledClock that updates the time once per second, but still increments
  time for subsequent calls
@patriknw patriknw force-pushed the wip-scheduled-clock-patriknw branch from c68897e to 6137ea3 Compare February 10, 2023 11:03
Copy link
Member

@johanandren johanandren left a comment

Choose a reason for hiding this comment

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

LGTM

@patriknw patriknw added this to the 2.8.0-M5 milestone Feb 16, 2023
@patriknw patriknw merged commit 7588b84 into main Feb 16, 2023
@patriknw patriknw deleted the wip-scheduled-clock-patriknw branch February 16, 2023 12:51
@pvlugter
Copy link
Contributor

👍🏼

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.

3 participants