Skip to content
This repository has been archived by the owner on Oct 31, 2022. It is now read-only.

Switch to using HashedWheelTimer #31

Merged
merged 1 commit into from Aug 21, 2018
Merged

Switch to using HashedWheelTimer #31

merged 1 commit into from Aug 21, 2018

Conversation

samstarling
Copy link
Owner

This stops the exception seen by @seanbrant in #24, by no longer using DefaultTimer.getInstance.

ERROR com.twitter.app.LoadService - LoadService: failed to instantiate 'com.samstarling.prometheusfinagle.PrometheusStatsReceiver' for the requested service 'com.twitter.finagle.stats.StatsReceiver'
java.lang.NullPointerException: null
	at com.twitter.util.ProxyTimer.schedulePeriodically(Timer.scala:148)
	at com.twitter.util.Timer.schedule(Timer.scala:56)
	at com.twitter.util.Timer.schedule$(Timer.scala:53)

@samstarling samstarling merged commit ce295d1 into master Aug 21, 2018
@samstarling samstarling deleted the hashed-wheel-timer branch August 21, 2018 13:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@samstarling