Skip to content

Conversation

@cedricve
Copy link
Member

@cedricve cedricve commented Jan 15, 2026

Description

Pull Request Description

Title: feature/add-analysis-queue-option

Motivation:
This pull request introduces an AnalysisQueue option to the RabbitMQ configuration. The addition of this new queue is motivated by the need for a dedicated queue to handle analysis messages separately from other types of messages in the system. By segregating analysis messages, we can better manage and prioritize these tasks, leading to improved performance and more organized message processing.

Changes:

  • Added AnalysisQueue field to the RabbitOptions struct.
  • Implemented the SetAnalysisQueue method in the RabbitOptionsBuilder to allow the configuration of the analysis queue name.

Why it improves the project:

  1. Enhanced Organization: Having a dedicated queue for analysis messages helps in organizing and managing different types of messages more effectively.
  2. Improved Performance: By segregating analysis tasks into their own queue, we can ensure that these tasks are processed independently and potentially more efficiently, without interference from other message types.
  3. Scalability: This change allows for better scalability as the system can now handle different workloads separately, making it easier to monitor and optimize each queue's performance individually.

Overall, this enhancement contributes to a more robust and maintainable message processing system within the project.

@codecov
Copy link

codecov bot commented Jan 15, 2026

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
pkg/queue/rabbitmq.go 0.00% 3 Missing ⚠️

📢 Thoughts on this report? Let us know!

@cedricve cedricve merged commit 4416bc6 into main Jan 15, 2026
6 checks passed
@cedricve cedricve deleted the feature/add-analysis-queue-option branch January 15, 2026 10:20
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