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

Add FetchAvailability result #519

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add FetchAvailability result #519

wants to merge 2 commits into from

Conversation

RichLogan
Copy link
Contributor

This is 50% a proposal and 50% a PR.

FETCH_OK requires some attributes that an implementation needs to provide:

End Of Track: 1 if all objects have been published on this track, so the Largest Group ID and Object Id indicate the last Object in the track, 0 if not.
Largest Group ID: The largest Group ID available for this track.
Largest Object ID: The largest Object ID available within the largest Group ID for this track.

This allows the FetchReceived implementor to provide them, if the fetch can be resolved.

A similar API will be needed for JoiningFetch (for matching subscription lookup that needs to be implementor provided), so wanted to float this separately.

@RichLogan
Copy link
Contributor Author

Given what was discussed today, will change this to be a more generic cache lookup callback.

@RichLogan RichLogan added the not ready Do not merge this pull request yet as it is not ready label Mar 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not ready Do not merge this pull request yet as it is not ready
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants