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

Take the actual sequence numbers, instead of relying on the underlying Seq #409

Merged
merged 2 commits into from Nov 29, 2018

Conversation

Projects
None yet
3 participants
@Fokko
Copy link
Contributor

Fokko commented Nov 10, 2018

For the tests we want to use the actual sequence number instead of using the size of the underlying Seq. In real world, the consumer group of the event hub might nog start at zero, but at a high number.

Take the actual sequence numbers, instead of relying on Seq
For the tests we want to use the actual sequence number instead of
using the size of the underlying Seq. In real world, the consumer
group of the event hub might nog start at zero, but at a high number

@Fokko Fokko changed the title Take the actual sequence numbers, instead of relying on Seq Take the actual sequence numbers, instead of relying on the underlying Seq Nov 10, 2018

@Fokko

This comment has been minimized.

Copy link
Contributor

Fokko commented Nov 10, 2018

Required by #408

@sabeegrewal sabeegrewal merged commit 2bdfa15 into Azure:master Nov 29, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
license/cla All CLA requirements met.

@sjkwak sjkwak added this to the 2.3.7 milestone Jan 3, 2019

@sjkwak sjkwak added the enhancement label Jan 3, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment