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

Publication cache re-implementation #64

Open
JEnoch opened this issue Jan 14, 2022 · 0 comments
Open

Publication cache re-implementation #64

JEnoch opened this issue Jan 14, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@JEnoch
Copy link
Member

JEnoch commented Jan 14, 2022

Currently a route for TRANSIENT_LOCAL Writers uses a zenoh-ext PublicationCache to store the published samples. However this cache is not designed to maintain 1 queue per-writer, leading to cache only 1 sample despite several Writer with history length 1 might be served by this route (see #61).

A new publication cache (specific to this plugin) must be implemented to support per-writer caching, with a queue size corresponding to each matching Writer. The cache management must be dynamic to allow addition/removal of queues on Writers discovery/undiscovery.
Ideally, we should also foresee the possibility to support per-instance queues, to be used when the plugin will be able to distinguish instances at samples reception.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants