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.

Sign up for GitHub

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

Jump to bottom

Make retry policy configurable #87

Closed
chbatey opened this issue Sep 7, 2015 · 1 comment
Closed

Make retry policy configurable #87

chbatey opened this issue Sep 7, 2015 · 1 comment
Milestone
0.6

Comments

@chbatey
Copy link
Collaborator

chbatey commented Sep 7, 2015

Persistent actors are stopped if a persist fails which can be an expensive operation as when the Actor is restarted a full reply is required.

Could start with a fixed retry policy + offer the built in ones from the datastax driver.
@magro
Copy link
Contributor

magro commented Dec 8, 2015

We're occasionally getting WriteTimeoutExceptions (e.g. "Cassandra timeout during write query at consistency LOCAL_QUORUM (2 replica were required but only 1 acknowledged the write") and therefore I'm thinking about adding retries (we've 100s of thousands events to replay).

Should we start with the FixedRetryPolicy and make "write-timeout-retries" configurable?

magro added a commit to magro/akka-persistence-cassandra that referenced this issue Dec 8, 2015
@magro
Retry failing write requests, add write-retries config (default=3).
1d30b10 
This is especially useful for PersistentActors with bigger journals /
more events to replay in case of write errors. But also for smaller
journals write retries are probably what a user might expect / want,
therefore the default=3 is chosen.

Refs krasserm#87
@magro magro mentioned this issue Dec 8, 2015
Merged
@patriknw patriknw added this to the 0.6 milestone Dec 9, 2015
@patriknw patriknw mentioned this issue Dec 9, 2015
Merged
rafalsiwiec pushed a commit to rafalsiwiec/akka-persistence-cassandra that referenced this issue Sep 15, 2016
@patriknw
Merge pull request krasserm#87 from rafalsiwiec/replay-queries-consis…
e9bfdef 
…tency-level

CassandraJournal uses cassandra-journal.read-consistency
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.6
Development

No branches or pull requests
4 participants
@magro @krasserm @patriknw @chbatey