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

Updates from CUMULUS-3940 #361

Merged
merged 2 commits into from
Jan 4, 2025
Merged

Updates from CUMULUS-3940 #361

merged 2 commits into from
Jan 4, 2025

Conversation

Jkovarik
Copy link
Member

Bringing in updates from nasa/cumulus#3890

@Jkovarik Jkovarik added the Needs Review Looking for a reviewer label Jan 2, 2025
@@ -10,6 +10,9 @@ The query uses the following optional parameters:
| --- | --- |
| bucket | The bucket to read records from. Defaults to the Core system bucket|
| path | The S3 prefix (path) to read DLQ records from. Defaults to `<stackName>/dead-letter-archive/sqs/`|
| batchSize | Specifies how many DLA objects to read from S3 and hold in memory. Defaults to 1000|
| concurrency | Specifies how many messages to process at the same time. Defaults to 30|
| dbMaxPool | specifies how many database connections to allow the process to utilize. Defaults to 30. Process should at minimum the value set for `concurrency`|
Copy link
Contributor

Choose a reason for hiding this comment

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

capitalize Specifies for line 15 then should be good 👍

Copy link
Member Author

Choose a reason for hiding this comment

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

😅 good catch! b9a499f

@Jkovarik Jkovarik removed the Needs Review Looking for a reviewer label Jan 3, 2025
Copy link
Contributor

@Nnaga1 Nnaga1 left a comment

Choose a reason for hiding this comment

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

Great work!! :D

@Jkovarik Jkovarik merged commit 45bf334 into master Jan 4, 2025
3 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