Skip to content

Fix "duplicate task" error #130

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 4 commits into from
Feb 7, 2024
Merged

Conversation

ted-selig-five9
Copy link
Contributor

Use milliseconds to avoid duplicate task name. I am getting The task cannot be created because a task with this name existed too recently. For more information about task de-duplication see https://cloud.google.com/tasks/docs/reference/rest/v2/projects.locations.queues.tasks/create#body.request_body.FIELDS.task. when running the app on version v3.6.0

use milliseconds to avoid duplicate task name
@ted-selig-five9 ted-selig-five9 changed the title Update CloudTasksQueue.php Fix "duplicate task" error Feb 6, 2024
@ted-selig-five9
Copy link
Contributor Author

@marickvantuil can I please get feedback on this? Basically my app has many concurrent requests so just adding the timestamp with seconds still can lead to creating duplicate names. So I want to suggest using timestamp with milliseconds. Thanks!

@marickvantuil marickvantuil added the safe-to-test Pull request has access to workflow secrets to run tests label Feb 6, 2024
@marickvantuil
Copy link
Member

Thanks for the PR! Can you check the failing tests? Then I can merge it and make a new version.

fix it_adds_a_task_name_based_on_the_display_name
@github-actions github-actions bot removed the safe-to-test Pull request has access to workflow secrets to run tests label Feb 6, 2024
fix retried_jobs_get_a_new_name
@ted-selig-five9
Copy link
Contributor Author

I made an attempt at a fix

@marickvantuil marickvantuil added the safe-to-test Pull request has access to workflow secrets to run tests label Feb 7, 2024
@github-actions github-actions bot removed the safe-to-test Pull request has access to workflow secrets to run tests label Feb 7, 2024
@marickvantuil marickvantuil added the safe-to-test Pull request has access to workflow secrets to run tests label Feb 7, 2024
@marickvantuil marickvantuil merged commit 0589e26 into stackkit:master Feb 7, 2024
@marickvantuil
Copy link
Member

Tagged v3.6.5!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe-to-test Pull request has access to workflow secrets to run tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants