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

chore: refactor listen_to handler as alisten_to and make POST v3/ async #57

Open
viniarck opened this issue Jul 19, 2023 · 0 comments
Open
Labels
priority_low Low priority tech_debt Tech debt

Comments

@viniarck
Copy link
Member

There's opportunity in the future to refactor the current decorated on_topology_event handler with alisten_to instead, and then also make the POST v3/ endpoint async, the main benefit would also be able to remove the self._lock since we wouldn't have tasks yields happening, while also being having liighter handlers, the roadblock to address this chore here is also refactor the unit tests issue #50 to be async compatible, until we get there, in the meantime, it's easier to leverage dynamic_single thread mode on listen_to instead kytos-ng/kytos#399.

@viniarck viniarck added priority_low Low priority tech_debt Tech debt labels Jul 19, 2023
@viniarck viniarck changed the title chore: refactor listen_to as alisten_to and make POST v3/ async chore: refactor listen_to handler as alisten_to and make POST v3/ async Jul 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority_low Low priority tech_debt Tech debt
Projects
None yet
Development

No branches or pull requests

1 participant