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

fixing issue of returning inclusive ledger offsets with completions #1932

Merged
8 commits merged into from
Jul 1, 2019

Conversation

gabor-aranyossy
Copy link
Contributor

@gabor-aranyossy gabor-aranyossy commented Jun 28, 2019

Fixed an issue when CompletionService returns offsets having inclusive semantics when used for re-subscription.

Pull Request Checklist

NOTE: CI is not automatically run on non-members pull-requests for security
reasons. The reviewer will have to comment with /AzurePipelines run to
trigger the build.

@gabor-aranyossy gabor-aranyossy force-pushed the completion_stream_offset_fix branch from 86c7a44 to 4291044 Compare June 28, 2019 09:20
@gabor-aranyossy gabor-aranyossy marked this pull request as ready for review June 28, 2019 09:20
@gabor-aranyossy gabor-aranyossy added the component/ledger Sandbox and Ledger API label Jun 28, 2019
unreleased.rst Outdated Show resolved Hide resolved
@mergify mergify bot closed this Jul 1, 2019
@mergify mergify bot deleted the completion_stream_offset_fix branch July 1, 2019 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/ledger Sandbox and Ledger API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants