Skip to content

Avoid dead letter for rebalance timeout msg #28274

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
Merged
merged 1 commit into from
Dec 5, 2019
Merged

Avoid dead letter for rebalance timeout msg #28274

merged 1 commit into from
Dec 5, 2019

Conversation

chbatey
Copy link
Contributor

@chbatey chbatey commented Nov 28, 2019

E.g.

Message [akka.actor.ReceiveTimeout$] from Actor[akka://KafkaToSharding/system/sharding/user-processingCoordinator/singleton/coordinator/$d#-955782993] to Actor[akka://KafkaToSharding/system/sharding/user-processingCoordinator/singleton/coordinator/$d#-955782993] was not delivered. [10] dead letters encountered, no more dead letters will be logged in next [5.000 min]. If this is not an expected behavior then Actor[akka://KafkaToSharding/system/sharding/user-processingCoordinator/singleton/coordinator/$d#-955782993] may have terminated unexpectedly. This logging can be turned off or adjusted with configuration settings 'akka.log-dead-letters' and 'akka.log-dead-letters-during-shutdown'.

@akka-ci akka-ci added the validating PR is currently being validated by Jenkins label Nov 28, 2019
@akka-ci akka-ci added tested PR that was successfully built and tested by Jenkins and removed validating PR is currently being validated by Jenkins labels Nov 28, 2019
@akka-ci
Copy link

akka-ci commented Nov 28, 2019

Test PASSed.

patriknw
patriknw approved these changes Dec 5, 2019
View reviewed changes
Copy link
Contributor

@patriknw patriknw 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 merged commit 2c96a57 into akka:master Dec 5, 2019
@patriknw patriknw added this to the 2.6.1 milestone Dec 5, 2019
zbynek001 added a commit to zbynek001/akka.net that referenced this pull request Mar 24, 2020
@zbynek001
Avoid dead letter for rebalance timeout msg
7c39308 
migrated from akka/akka#28274
@zbynek001 zbynek001 mentioned this pull request Mar 24, 2020
Merged
zbynek001 added a commit to zbynek001/akka.net that referenced this pull request Mar 26, 2020
@zbynek001
Avoid dead letter for rebalance timeout msg
24dd919 
migrated from akka/akka#28274
Aaronontheweb added a commit to akkadotnet/akka.net that referenced this pull request Mar 27, 2020
@zbynek001 @Aaronontheweb
Sharding update (#4354)
735aafb 
* Avoid dead letter for rebalance timeout msg

migrated from akka/akka#28274

* Reconsider cluster.role.<role-name>.min-nr-of-members fallback
migrated from akka/akka#28203

* Use dedicated message for PersistentShardCoodinator termination instead of PoisonPill
migrated from akka/akka#28104

* Don't initialize durable keys when rememberEntities not used
migrated from akka/akka#28568

* Improve error message when coordinator not found
migrated from akka/akka#28576

* Shard region registration to more potential oldest

* Keep track of Leaving and Exiting members in ShardRegion and
  attempt to register to coordinator at several of the oldest if
  they have status Leaving and Exiting. Include all up to and
  including the first member with status Up.
* Sending to wrong node doesn't matter, will be suppressed deadLetter.
* Same for the GracefulShutdownReq which already had that intention by
  sending to 2 oldest.

migrated from akka/akka#28470

* Log non-completed rebalance at warning level
migrated from akka/akka#28335

* api approval fix

* review fixes

Co-authored-by: Aaron Stannard <aaron@petabridge.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@patriknw patriknw patriknw approved these changes

@johanandren johanandren johanandren approved these changes

Assignees
No one assigned
Labels
tested PR that was successfully built and tested by Jenkins
Projects
None yet
Milestone
2.6.1
Development

Successfully merging this pull request may close these issues.
4 participants
@chbatey @akka-ci @patriknw @johanandren