Added support for MultiGetSlice for Counter columns and super columns #101

Merged
merged 4 commits into from Jan 6, 2013

Projects

None yet

2 participants

@Aaronontheweb
Contributor

Added support for MultiGetSlice for Counter columns and super columns

Quick question though: I haven't been able to get any of the unit tests that connect to a live Cassandra instance to run due to errors with the CassandraDatabaseSetup.cs bootstrapper throwing errors - I think it has to do with the version of Cassandra running on my machine (DataStax Community Edition v1.1)

Could someone try running these and see if my new unit test Counter_GetSlice_Columns() passes? I've been able to verify that the behavior works as expected inside my application now that I've made this change, but I don't like the fact that I can't get 80ish of the built-in unit tests to run on my system :(

Aaronontheweb added some commits Jan 6, 2013
@Aaronontheweb Aaronontheweb Changed the MultiGetColumnFamilySlice operator to check to see if a c…
…ounter column type was returned, and do the subsequent conversion if found. Otherwise, default to normal behavior.
668cc47
@Aaronontheweb Aaronontheweb Made the same change to SuperColumnFamilies cce150d
@Aaronontheweb Aaronontheweb Added a CounterColumnFamily to the DatabaseSetup a6e3f52
@Aaronontheweb Aaronontheweb Finished adding the Counters bootstrap information to the CassandraDa…
…tabaseSetup.cs file and implemented the Counter_GetSlice_Columns() unit test in the MultiSliceGetTest.cs file
311cd7b
Contributor
nberardi commented Jan 6, 2013

These look good, but please be respectful of the tab formatting. If you don't have this tool yet. It is great for fixing mixed tab/space files.

http://blogs.msdn.com/b/visualstudio/archive/2012/11/07/productivity-power-tools-2012.aspx

@nberardi nberardi merged commit 8825803 into fluentcassandra:master Jan 6, 2013
Contributor

will do - thanks @nberardi

@nberardi nberardi was assigned Jan 14, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment