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

WIFI-13857: fix: modified code to use flush() when internal queue is not loaded #362

Merged
merged 1 commit into from
Jun 19, 2024

Conversation

i-chvets
Copy link
Contributor

@i-chvets i-chvets commented Jun 19, 2024

Description

When queue is not loaded, use of poll() might prevent Kafka messages to be delivered on time. This can affect Kafka registration messages. The use of flush() on lightly loaded queue should allow messages to be delivered.

Details in:
https://telecominfraproject.atlassian.net/browse/WIFI-13597
https://telecominfraproject.atlassian.net/browse/WIFI-13857

Summary of changes

  • Modified code to use flush() when internal queue is not loaded.

i-chvets added a commit to Telecominfraproject/wlan-cloud-owprov that referenced this pull request Jun 19, 2024
@i-chvets i-chvets changed the title fix: modified code to use flush() when internal queue is not loaded WIFI-13857: fix: modified code to use flush() when internal queue is not loaded Jun 19, 2024
i-chvets added a commit to Telecominfraproject/wlan-cloud-ucentralsec that referenced this pull request Jun 19, 2024
i-chvets added a commit to Telecominfraproject/wlan-cloud-analytics that referenced this pull request Jun 19, 2024
Copy link
Contributor

@carsten989 carsten989 left a comment

Choose a reason for hiding this comment

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

May be moving the problem to a more difficult to diagnose level with this, but hopefully it'll solve the problem at hand.

@i-chvets i-chvets merged commit 5d89107 into master Jun 19, 2024
3 of 4 checks passed
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