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

Lack of available read handlers for a read should respond with BUSY, not RESOURCE_EXHAUSTED #18343

Closed
bzbarsky-apple opened this issue May 11, 2022 · 2 comments · Fixed by #18575
Assignees
Labels
spec Mismatch between spec and implementation V1.0

Comments

@bzbarsky-apple
Copy link
Contributor

Problem

If I do a read and get back RESOURCE_EXHAUSTED, there is no way to tell whether that's because there was another read in progress (in which case I should just retry) or whether it's because my read attempt used too many paths, filters, etc (in which case I need to change what sort of request I am sending).

Proposed Solution

Use BUSY to indicate "no available read clients right now", not RESOURCE_EXHAUSTED.

@bzbarsky-apple bzbarsky-apple added V1.0 spec Mismatch between spec and implementation labels May 11, 2022
@bzbarsky-apple
Copy link
Contributor Author

@bzbarsky-apple bzbarsky-apple self-assigned this May 17, 2022
@bzbarsky-apple
Copy link
Contributor Author

https://github.com/CHIP-Specifications/connectedhomeip-spec/issues/5198 is now fixed. So this just needs code changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec Mismatch between spec and implementation V1.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant