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

[ReadHandler] Analyze Max Timestamp recalculation behavior. #27996

Open
lpbeliveau-silabs opened this issue Jul 17, 2023 · 2 comments
Open

[ReadHandler] Analyze Max Timestamp recalculation behavior. #27996

lpbeliveau-silabs opened this issue Jul 17, 2023 · 2 comments
Labels
icd Intermittently Connected Devices
Milestone

Comments

@lpbeliveau-silabs
Copy link
Contributor

lpbeliveau-silabs commented Jul 17, 2023

Once the Now Timestamps are unified, we need to ensure the current behavior to ignore max in the past won't lead to reports being abandoned by mistake.

@lpbeliveau-silabs
Copy link
Contributor Author

@bzbarsky-apple
Copy link
Contributor

Things to think about carefully:

  1. The scenario described in those comments, with two handlers, one finishes reporting, we re-schedule things, etc.
  2. A scenario where a handler is overdue but we did not schedule a run yet (because we are running late) and a new handler is added.
  3. What other cases?

@lpbeliveau-silabs lpbeliveau-silabs added the icd Intermittently Connected Devices label Jul 18, 2023
@woody-apple woody-apple added this to the 1.3 milestone Jul 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
icd Intermittently Connected Devices
Projects
Development

No branches or pull requests

4 participants