Skip to content
This repository has been archived by the owner on Jul 22, 2020. It is now read-only.

ConsistencyLevel is configurable #46

Merged

Conversation

smiklosovic
Copy link
Contributor

Summary

Completes feature #20

Related tickets: #20

Consistency level is configurable in KeyspaceConfiguration via setter. In case we are in clustered environment, consistency level is set to ALL, ONE otherwise. If consistency is not set in KeyspaceConfiguration, previously resolved consistency level is set, otherwise level in KeyspaceConfiguration is used.


Pull Request (PR) Checklist

Code Review

  • Self code review

Tests

  • All tests passes

@smiklosovic
Copy link
Contributor Author

@hhandoko could you review please?

@smiklosovic
Copy link
Contributor Author

once merged, could you, preferably, release 0.11 please?

@hhandoko
Copy link
Owner

Thanks for your contribution @smiklosovic , much appreciated.

I'm planning another release for end of March (timeboxing for releases every two month). Can you work with SNAPSHOT release until then? Please let me know, we can discuss.

@hhandoko hhandoko merged commit 3cbce02 into hhandoko:master Feb 28, 2017
@smiklosovic
Copy link
Contributor Author

smiklosovic commented Feb 28, 2017

@hhandoko

Thanks for the merge. We are using this in production like environment and it would be nice not to depend on snapshots. End of March is too late for us unfortunately.

@hhandoko
Copy link
Owner

It's OK, I understand. I'll organise a patch-level release. Will let you know more tomorrow.

@hhandoko
Copy link
Owner

hhandoko commented Mar 2, 2017

@smiklosovic I've published 0.10.1, please keep a lookout in Maven Central, should sync within a day or two.

@smiklosovic smiklosovic deleted the feature/configurable_consistency_level branch March 2, 2017 09:48
@smiklosovic
Copy link
Contributor Author

Many thanks. Cheers

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants