Skip to content

Conversation

Arkatufus
Copy link
Contributor

@Arkatufus Arkatufus commented Jun 10, 2025

Fixes #7264

Changes

  • Add ToString() overrides for IScheduledTellMsg implementations, basically allows users get some insight into which scheduled messages were being dead lettered.

Turns

[WARNING][06/24/2024 14:51:03.026Z][Thread 0015][akka://ClusterClientDiscoverySpec/user/$b] DeadLetter from [akka://ClusterClientDiscoverySpec/user/$b#1239270490] to [akka://ClusterClientDiscoverySpec/user/$b#1239270490]: <Received dead letter from [akka://ClusterClientDiscoverySpec/user/$b#1239270490]: Akka.Actor.Scheduler.ScheduledTellMsg>

into

[WARNING][06/10/2025 17:28:52.161Z][Thread 0022][akka://ClusterClientDiscoverySpec/user/client1/$b] DeadLetter from [akka://ClusterClientDiscoverySpec/user/client1/$b#1665680760] to [akka://ClusterClientDiscoverySpec/user/client1/$b#1665680760]: <Received dead letter from [akka://ClusterClientDiscoverySpec/user/client1/$b#1665680760]: {ScheduledTellMsg: {Message: Akka.Cluster.Tools.Client.ClusterClient+ReconnectTimeout}}>

Copy link
Member

@Aaronontheweb Aaronontheweb left a comment

Choose a reason for hiding this comment

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

LGTM

@Aaronontheweb Aaronontheweb enabled auto-merge (squash) June 10, 2025 17:42
@Aaronontheweb Aaronontheweb disabled auto-merge June 10, 2025 17:47
@Aaronontheweb Aaronontheweb merged commit d4e571b into akkadotnet:dev Jun 10, 2025
3 of 10 checks passed
@Arkatufus Arkatufus deleted the Improve-ScheduledTellMsg-logging branch June 20, 2025 14:10
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.

Logs for ScheduledTellMsg DeadLetter are not specific enough
2 participants