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

[Merged by Bors] - fix consuming with multiple segments #1700

Closed
wants to merge 16 commits into from

Conversation

sehz
Copy link
Contributor

@sehz sehz commented Sep 29, 2021

resolves #1568

  • Remove unneeded spawn handle in Stream Fetch Handler
  • Fix Storage segment list search
  • Fix segmentload loading
  • Properly update end offset when active segment is rollover

@sehz sehz marked this pull request as draft September 29, 2021 05:44
@sehz sehz marked this pull request as ready for review September 30, 2021 05:04
@sehz
Copy link
Contributor Author

sehz commented Sep 30, 2021

bors r+

bors bot pushed a commit that referenced this pull request Sep 30, 2021
resolves #1568

* Remove unneeded spawn handle in Stream Fetch Handler
* Fix Storage segment list search
* Fix segmentload loading
* Properly update end offset when active segment is rollover
@bors
Copy link

bors bot commented Sep 30, 2021

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title fix consuming with multiple segments [Merged by Bors] - fix consuming with multiple segments Sep 30, 2021
@bors bors bot closed this Sep 30, 2021
@sehz sehz deleted the large_consume branch December 1, 2021 04:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant