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

Akka Source add supervision configuration #378

Open
KannarFr opened this issue Dec 16, 2021 · 3 comments
Open

Akka Source add supervision configuration #378

KannarFr opened this issue Dec 16, 2021 · 3 comments

Comments

@KannarFr
Copy link
Member

The consumer used in the pulsar4s Akka Source is closed and stops the stream for every encountered error.

I think we should use the supervision decider optionally passed to the flow. I don't know if it's possible yet, if not we should have a way to pass a partial function to define the behavior to have during exception on consumption.

I am particularly interested in parsing issues during consumption. I would like to Skip the message and continue to the Next message. So Supervision.Resume looks good.

@KannarFr
Copy link
Member Author

@gmethvin WDYT?

@KannarFr
Copy link
Member Author

nvm, ConsumerMessage[T].valueTry should be enough.

@gmethvin
Copy link
Collaborator

The parsing errors are already exposed with valueTry but there might be others where it would make sense to use a supervision decider.

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

No branches or pull requests

2 participants