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

refactor: remove Subscriber interface #37

Closed
ar3s3ru opened this issue Aug 23, 2021 · 1 comment · Fixed by #42, #44 or #46
Closed

refactor: remove Subscriber interface #37

ar3s3ru opened this issue Aug 23, 2021 · 1 comment · Fixed by #42, #44 or #46
Labels
enhancement New feature or request
Milestone

Comments

@ar3s3ru
Copy link
Collaborator

ar3s3ru commented Aug 23, 2021

The eventstore.Subscriber interface is used to support push semantics of Event Processors: new Events are pushed from the Event Store to the Processor, similar to Pub/Sub approach.

Push semantics is not always supported by all the event streaming technologies out there, and generally speaking event streaming platforms prefer pull semantics instead (currently supported by the eventstore.Streamer interface).

We can remove any trace of this interface from the codebase for v0.1.0, and only rely on pull semantics. This makes things harder for Volatile Subscriptions, but we could add some support for the Stream* methods to fetch events from the latest one available.

@ar3s3ru
Copy link
Collaborator Author

ar3s3ru commented Aug 30, 2021

Closing this, the Volatile subscription effort is now tracked in #51

@ar3s3ru ar3s3ru closed this as completed Aug 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment