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

Change "Kafka commit is to be retried" message to show it's cause #1468

Merged
merged 1 commit into from
Feb 1, 2022
Merged

Change "Kafka commit is to be retried" message to show it's cause #1468

merged 1 commit into from
Feb 1, 2022

Conversation

destrim
Copy link

@destrim destrim commented Feb 1, 2022

Changes due to lack of information about what caused "Kafka commit is to be retried" message.

Before:
image

After:
image

Btw is there any solution or configuration for spamming "Kafka commit is to be retried" logs? Maybe reducing the frequency or muting some of them.

@lightbend-cla-validator
Copy link

At least one pull request committer is not linked to a user. See https://help.github.com/en/articles/why-are-my-commits-linked-to-the-wrong-user#commits-are-not-linked-to-any-user

Copy link
Member

@ennru ennru left a comment

Choose a reason for hiding this comment

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

LGTM.

@ennru ennru added this to the 3.0.1 milestone Feb 1, 2022
@ennru ennru merged commit b97f090 into akka:master Feb 1, 2022
@ennru
Copy link
Member

ennru commented Feb 1, 2022

Thank you! Very helpful to understand what is wrong indeed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants