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

Added Togglz - Cassandra integration #128

Merged
merged 1 commit into from May 9, 2015
Merged

Added Togglz - Cassandra integration #128

merged 1 commit into from May 9, 2015

Conversation

@artur-tamazian
Copy link

@artur-tamazian artur-tamazian commented May 7, 2015

StateRepository implementation that uses Astyanax to store features’
states in Cassandra.

@atfire atfire force-pushed the atfire:master branch 2 times, most recently from ac59600 to 3e7ee4b May 7, 2015
StateRepository implementation that uses Astyanax to store features’
states in Cassandra.
@atfire atfire force-pushed the atfire:master branch from 3e7ee4b to fe2ac69 May 7, 2015
@chkal
Copy link
Member

@chkal chkal commented May 7, 2015

Awesome! Thanks a lot. I hope I'll fine some time at the weekend to review it.

@chkal
Copy link
Member

@chkal chkal commented May 9, 2015

Thanks a lot. This module looks good. I cannot give it a test drive but I guess you did. ;)

Many many thanks for this contribution.

chkal added a commit that referenced this pull request May 9, 2015
Added Togglz - Cassandra integration
@chkal chkal merged commit e54ce94 into togglz:master May 9, 2015
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@chkal
Copy link
Member

@chkal chkal commented May 9, 2015

BTW: I'm not very faimilar with Cassandra. Is there a way to add unit/integration tests for this repository?

@artur-tamazian
Copy link
Author

@artur-tamazian artur-tamazian commented May 12, 2015

I can try to write a test using https://github.com/jsevellec/cassandra-unit. WIll try to do it this week

@chkal
Copy link
Member

@chkal chkal commented May 12, 2015

Great! That would be awesome! ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.