-
Notifications
You must be signed in to change notification settings - Fork 179
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
Use tokio to replace async_std for plugins and backends. #1264
Conversation
@wyfo do you mind reviewing this PR? |
BTW, if we decide to use another static Tokio runtime in the plugin and consequently remove the async-std dependency in zenoh-util, we can eliminate the outdated TimedEvent since it was used before Tokio-utils was available. |
55a4085
to
cb22e55
Compare
We need to double check first wether |
A quick search over our eclipse project shows we use |
A broader discussion should happen on wether every plugin has its own executor or a common executor can be used when statically linking the plugins. |
Discussion happened on Discord and there is a common agreement to go for a single runtime in case of static linking. To deal with static vs dynamic linking and symbols re-export |
62e5baf
to
12cfbce
Compare
Signed-off-by: ChenYing Kuo <evshary@gmail.com>
Signed-off-by: ChenYing Kuo <evshary@gmail.com>
12cfbce
to
993b510
Compare
Signed-off-by: ChenYing Kuo <evshary@gmail.com>
993b510
to
6d6f335
Compare
Related to #1257