Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

docs: update fullrt docs #768

Merged
merged 2 commits into from
Feb 5, 2025
Merged

Conversation

aschmahmann
Copy link
Contributor

@MichaelMure this has taken some time for me to put out since our conversation, but does this clear things up? Do you have other questions, thoughts, concerns?

@MichaelMure
Copy link
Contributor

LGTM, maybe clarify what's happening when publishing with a non-ready client? Does that means it reaches less accurate target nodes? More errors?

@MichaelMure
Copy link
Contributor

Also, are we expected to use that client in large sequential batch? What happens with concurrent Provide* calls? Is there a recommended batch size?

@BigLep BigLep added this to the Best Effort Track milestone Jun 3, 2022
@guillaumemichel guillaumemichel merged commit 87c65fa into master Feb 5, 2025
9 checks passed
@guillaumemichel guillaumemichel deleted the docs/fullrt-clarifications branch February 5, 2025 13:10
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants