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.

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

Refactored Message Queue and more #76

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from
Open

Refactored Message Queue and more #76

wants to merge 5 commits into from

Conversation

hrushikeshm-g
Copy link
Contributor

Title : Refactored Message Queue and more

✨ Changes ✨

✅ Refactored message-queue module and test cases.
✅ Refactored message-queue logger and strategies module and test cases.
✅ Added exponential backoff mechanism on server and worker side to prevent worker pinging.
✅ Improved code quality and maintainability.
✅ pull from dev

@hrushikeshm-g hrushikeshm-g requested review from bronevet-abc and a team as code owners February 5, 2025 11:17
bronevet-abc
bronevet-abc approved these changes Feb 6, 2025
View reviewed changes
py/exp/analsysis_helpers.py Outdated Show resolved Hide resolved
sight_service/message_queue/interface.py Outdated Show resolved Hide resolved
sight_service/single_action_optimizer.py Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bronevet-abc bronevet-abc bronevet-abc approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hrushikeshm-g @bronevet-abc