Skip to content

Increase rabbitMQ consumer timeout to one hour. #745

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

Merged
merged 1 commit into from
Apr 19, 2024

Conversation

BenGalewsky
Copy link
Contributor

During IDAP 200Gb challenge testing we found that very large datasets can take longer than 30 minutes to report from the DID finder back to the app. There is a default setting of 1/2 hour for a message to be ack-ed before closing the channel and putting the message back in the queue.

Increasing this value to 1 hour to avoid this problem.

During IDAP 200Gb challenge testing we found that very large datasets
can take longer than 30 minutes to report from the DID finder back to
the app. There is a default setting of 1/2 hour for a message to be ack-ed
before closing the channel and putting the message back in the queue.

Increasing this value to 1 hour to avoid this problem.
@BenGalewsky BenGalewsky requested a review from ivukotic April 19, 2024 15:16
@ivukotic
Copy link
Member

While this would prevent the issue we had last night (it took ~45 minutes), we did only one dataset that has 64k files. We will need to process 210k+ dataset files so just changing to 60 min won't really help.
My PR that sends x10 more files per batch helps, but we should also make REST API faster. It is way too slow even with 5 pods each with 5 workers. We need to find what bottlenecks it.

@BenGalewsky
Copy link
Contributor Author

I'm going to merge this just so we can find this later if it comes up again

@BenGalewsky BenGalewsky merged commit e0d5fa1 into develop Apr 19, 2024
@BenGalewsky BenGalewsky deleted the increase-consumer-timeout branch April 19, 2024 15:54
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