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

Clarify that a blocked channel will discard results #129

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

codesoap
Copy link

I'd like to suggest a clarification in the documentation of the Query and QueryContext functions, because I had misunderstood it. Before this PR, the documentation of both functions stated the following:

[...] The results are streamed to a channel. Sends will not block, so clients should make sure to either read or buffer.

When using the library, I made sure that the entries were indeed read from the channel in a timely manner, but some results were sporadically missing. After reading the code of the library, I realized that results are immediately discarded, if the receiving side is not quite as fast as the sending side. I think the documentation should put more emphasis on this behavior.

The documentation previously did not clarify, that results are
immediately discarded, if the receiving channel is not accepting
entries. This is now emphasized and a remedy is suggested more directly.
@codesoap codesoap requested review from a team as code owners February 10, 2025 18:33
Copy link

hashicorp-cla-app bot commented Feb 10, 2025

CLA assistant check
All committers have signed the CLA.

Copy link

CLA assistant check

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement

Learn more about why HashiCorp requires a CLA and what the CLA includes

Have you signed the CLA already but the status is still pending? Recheck it.

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 this pull request may close these issues.

1 participant