Skip to content

docs: update fullrt docs #768

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

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
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.

4 participants