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.

Sign up for GitHub

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

Jump to bottom

[FEATURE REQ] Detect sync function in async method #1207

Open
ChenTanyi opened this issue Jun 9, 2020 · 1 comment
Open

[FEATURE REQ] Detect sync function in async method #1207

ChenTanyi opened this issue Jun 9, 2020 · 1 comment
Assignees
@ChenTanyi
Labels
Mgmt Issues that refer to the management plane v2

Comments

@ChenTanyi
Copy link
Contributor

Due to reactor design, we should not use sync function in async method.
@ChenTanyi ChenTanyi added the Mgmt Issues that refer to the management plane label Jun 9, 2020
@triage-new-issues triage-new-issues bot removed the triage label Jun 9, 2020
@ChenTanyi ChenTanyi added the v2 label Jun 9, 2020
@yungezz yungezz added the v2 GA v2 GA candidate label Jul 22, 2020
@yungezz yungezz removed the v2 GA v2 GA candidate label Aug 12, 2020
@yungezz
Copy link
Member

yungezz commented Aug 12, 2020

synced with @ChenTanyi , known cases are fixed already. So deprioritize this one from GA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ChenTanyi ChenTanyi

Labels
Mgmt Issues that refer to the management plane v2
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ChenTanyi @yungezz