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

RetrySynchronizationManager cause my context lost #355

Open
zyx13131133 opened this issue May 31, 2023 · 1 comment
Open

RetrySynchronizationManager cause my context lost #355

zyx13131133 opened this issue May 31, 2023 · 1 comment

Comments

@zyx13131133
Copy link

RetrySynchronizationManager use ThreadLocal. when use @async cooperate @retryable Will cause the context data I set myself to be lost. Can open an interface for setting up the Theadlocal type

@artembilan
Copy link
Member

It does not make sense to apply the retry on an async method: we just would eliminate the purpose of an async nature to get a gain from the retry - essentially blocking that async call to catch a possible exception from there.
Therefore your request is not clear: we need more details and clarification how that suppose to work without breaking any original goals of your services design.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants