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

CassandraSinkCluster: Document desired error handling #875

Merged
merged 4 commits into from Oct 26, 2022

Conversation

rukai
Copy link
Member

@rukai rukai commented Oct 21, 2022

This documents what shotover's error handling for CassandraSinkCluster should look like.
Currently we dont handle in the way described here, but I wanted us to agree on an error handling approach that seemed reasonable and that we can work towards. We can of course change things later if we find we need to though.

We could also return an overloaded error when a connection error occurs before sending the message as that would simplify shotover by letting us push retries onto the client.

docs/src/transforms.md Outdated Show resolved Hide resolved
@rukai rukai enabled auto-merge (squash) October 26, 2022 04:30
@rukai rukai merged commit 28f4306 into shotover:main Oct 26, 2022
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.

None yet

4 participants