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

Android - tweaks to heartbeat collection subscription (part 1) #74

Open
bplattenburg opened this issue May 10, 2024 · 2 comments · May be fixed by #85
Open

Android - tweaks to heartbeat collection subscription (part 1) #74

bplattenburg opened this issue May 10, 2024 · 2 comments · May be fixed by #85
Assignees

Comments

@bplattenburg
Copy link
Member

bplattenburg commented May 10, 2024

Today, we automatically subscribe any device using the hearbeat tool to the entire dittotools_devices collection, but that doesn't scale to production across multiple locations.

We either need to provide a way to disable this subscription (which has the downside of breaking multihop for a device which can't see the big peer e.g. lost LAN but still has BLE) or provide the ability to scope it to a more limited query like "devices at the same locationId as this device"

Note - we should also pair any limited query here with a matching eviction query to cover devices changing location and other edge cases

@bplattenburg
Copy link
Member Author

In the short term, the path forward here is to use publishToDittoCollection in the heartbeat config to control whether to subscribe to the collection.

Longer term, I'll break out a story (link to be added here) to narrow the subscription to something like "devices in the same location" to allow for flooding & multihop to get data out without pulling data in from remote devices

@bplattenburg bplattenburg changed the title Android - tweaks to heartbeat collection subscription Android - tweaks to heartbeat collection subscription (part 1) May 31, 2024
@bplattenburg
Copy link
Member Author

Part 2 (location-specific subscriptions) broken out into #84

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

Successfully merging a pull request may close this issue.

2 participants