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

Error Recovery #49

Closed
sandersdan opened this issue Apr 17, 2020 · 1 comment
Closed

Error Recovery #49

sandersdan opened this issue Apr 17, 2020 · 1 comment

Comments

@sandersdan
Copy link
Contributor

After an error in processing (configure, decode, or otherwise), there are multiple potential ways we could allow recovery:

  • Automatically recover at the next keyframe.
  • Recover after the next configure().
  • Recover after the next reset().
  • The above, but discard (abort) all queued requests first.
  • Do not allow recovery at all.

The latter two are easier to reason about, but some apps would be simpler with a different choice. Perhaps it should be configurable.

(Example that is difficult to reason about: if a configure() fails or is aborted, what configuration would we use to decode the next keyframe? This may be a reason to require configure() after reset()--it is always unambiguous.)

@chcunningham
Copy link
Collaborator

Do not allow recovery at all.

This is what we landed on, as it is simplest both conceptually and from an interface POV.

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

No branches or pull requests

2 participants