-
Notifications
You must be signed in to change notification settings - Fork 135
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
testing with AWS Keyspaces #866
Conversation
@@ -140,7 +140,7 @@ akka.persistence.cassandra { | |||
|
|||
# Maximum number of messages that will be batched when using `persistAsync`. | |||
# Also used as the max batch size for deletes. | |||
max-message-batch-size = 100 | |||
max-message-batch-size = 30 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Batches with more than 30 statements are not supported
# the journal does not use any counters or collections | ||
default-idempotence = true | ||
} | ||
} | ||
akka-persistence-cassandra-snapshot-profile { | ||
basic.request { | ||
consistency = ONE | ||
consistency = LOCAL_QUORUM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it should be possible to use LOCAL_ONE for the reads but fails so might be required for the writes.
Added a bunch of changes:
I opted for reusing the keyspace because "create tables" is faster than "create a keyspace, and then the tables" so I shaved 20 seconds from each test run. Tried to keep the ratio: "1 commit :: 1 test fixed" |
Just to show what changes I had to do to run tests with Amazon Keyspaces
See test results in #865 (comment)