Switch to lowercase keyspace name #36

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
1 participant
Contributor

jnunemaker commented Nov 8, 2012

Must be my version of cassandra (1.1.5), but I could not get the tests to run without changing to a lowercase keyspace name.

I tried just doing the include check with lowercase, but cassandra really did not like the switching of case in various places. This seems to work best from what I can tell.

Decided rather than just ask you about it, I would start the discussion with a version that seems to fix the problem for me and get your thoughts.

If you want, I can send you the output I get.

@jnunemaker jnunemaker Switch to lowercase keyspace name.
Must be my version of cassandra (1.1.5), but I could not get the tests to run without changing to a lowercase keyspace name.

I tried just doing the include check with lowercase, but cassandra really did not like the switching of case in various places. This seems to work best from what I can tell.
a0ab016
Contributor

jnunemaker commented Nov 8, 2012

Of course as soon as I send this, it starts passing with uppercase for me. Hmm... not sure what kind of state I got into that it failed over and over. I'll close this for now until I come across it again and can re-create. Sorry for the noise.

jnunemaker closed this Nov 8, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment