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 an option to allow resuming a paused partition when subscribing #75

Merged

Conversation

Jmgr
Copy link
Contributor

@Jmgr Jmgr commented Aug 28, 2020

Liftbridge offers the option of pausing a stream (with some or all of its partitions) to reduce the amount of CPU and memory usage on idle streams. When that happens, every subscriber to a partition receives an error. Publishing to a paused partition will resume it, allowing it to function as before.

Our use-case includes a situation where we would like to read from a partition even if it is paused. It has to be resumed to do so, but we do not want to publish to it. This PR adds an option to the subscription request to allow it to resume a partition instead of returning an error.

@tylertreat
Copy link
Member

lgtm, just need to update liftbridge-api dep.

@Jmgr
Copy link
Contributor Author

Jmgr commented Sep 1, 2020

The CI build seems to fail because the latest liftbridge-api has v2 RPCs, which triggers a failure go-liftbridge's v1 tests. Not sure what to do here. Should there be a v2 also for liftbridge-api?

@tylertreat tylertreat mentioned this pull request Sep 1, 2020
@tylertreat
Copy link
Member

Fixed in #77

@tylertreat tylertreat merged commit 618b3b5 into liftbridge-io:master Sep 1, 2020
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.

2 participants