Skip to content

Add callback id to long callback key generation. - #2473

Merged
T4rk1n merged 6 commits into
devfrom
fix-2221
Mar 29, 2023
Merged

T4rk1n merged 6 commits into
devfrom
fix-2221

Conversation

@T4rk1n

@T4rk1n T4rk1n commented Mar 21, 2023

Copy link
Copy Markdown
Contributor

Fix #2221

@T4rk1n
T4rk1n requested a review from alexcjohnson as a code owner March 21, 2023 18:59

@chriddyp chriddyp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

💃🏼 once a test is added

@T4rk1n

T4rk1n commented Mar 28, 2023

Copy link
Copy Markdown
Contributor Author

Seems like there is some issue with the celery test, not sure if the solution here is enough for celery or if it's the test that is flawed, only one function is kept for both outputs in the worker and it's also random which one is choosen.

@T4rk1n
T4rk1n merged commit 6773440 into dev Mar 29, 2023
@T4rk1n
T4rk1n deleted the fix-2221 branch March 29, 2023 14:39
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.

[BUG] Background callbacks with different outputs not working

2 participants