Skip to content

fix: Hotfix for batch_add_requests batch size limit#261

Merged
janbuchar merged 4 commits into
masterfrom
hotfix-batch-add-requests
Sep 5, 2024
Merged

fix: Hotfix for batch_add_requests batch size limit#261
janbuchar merged 4 commits into
masterfrom
hotfix-batch-add-requests

Conversation

@janbuchar

Copy link
Copy Markdown
Contributor

No description provided.

@janbuchar janbuchar added adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team. labels Sep 4, 2024
@janbuchar janbuchar requested review from B4nan and vdusek September 4, 2024 12:24
@github-actions github-actions Bot added this to the 97th sprint - Tooling team milestone Sep 4, 2024

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Pull Request Tookit has failed!

If issue is not linked to the pull request then estimate the pull request!

Comment thread src/apify/apify_storage_client/_request_queue_client.py Outdated
@janbuchar janbuchar marked this pull request as draft September 4, 2024 13:34
@janbuchar

Copy link
Copy Markdown
Contributor Author

This will need an integration test... Plus the API rejects duplicate uniqueKeys in the same batch.

@github-actions github-actions Bot added the tested Temporary label used only programatically for some analytics. label Sep 5, 2024
@janbuchar janbuchar requested a review from vdusek September 5, 2024 11:49
@janbuchar janbuchar marked this pull request as ready for review September 5, 2024 11:49
@janbuchar janbuchar merged commit 61d7a39 into master Sep 5, 2024
@janbuchar janbuchar deleted the hotfix-batch-add-requests branch September 5, 2024 12:22
vdusek added a commit that referenced this pull request Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team. tested Temporary label used only programatically for some analytics.

2 participants