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

[kraken]: warn only when certain of the number of RT messages in queue #3993

Merged
merged 1 commit into from
May 3, 2023

Conversation

pbougue
Copy link
Contributor

@pbougue pbougue commented May 3, 2023

As the number of prefetched message is unsure when no ready message is available, avoid logging in this case.

improvement after #3968

fix https://github.com/hove-io/navitia-docker-compose/issues/153

As the number of prefetched message is unsure when no ready message is
available, avoid logging in this case.

improvement after #3968

fix https://github.com/hove-io/navitia-docker-compose/issues/153
@pbougue pbougue requested review from kadhikari and pbench May 3, 2023 14:17
@pbougue pbougue marked this pull request as ready for review May 3, 2023 14:17
@sonarqubecloud
Copy link

sonarqubecloud bot commented May 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

11.1% 11.1% Coverage
0.0% 0.0% Duplication

@pbougue pbougue merged commit da263ac into dev May 3, 2023
@pbougue pbougue deleted the smarter_warning branch May 3, 2023 16:33
@kadhikari kadhikari changed the title [kraken] warn only when certain of the number of RT messages in queue [kraken]: warn only when certain of the number of RT messages in queue May 9, 2023
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.

2 participants