-
Notifications
You must be signed in to change notification settings - Fork 140
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
renamed Cassandra into CassandraOld #217
base: master
Are you sure you want to change the base?
Conversation
@psanford Any comments/thoughts about this? |
I agree that it would be useful to be able to use both gems at the same time. However, it seems like doing this will break all existing users of the gem. |
Agree, then bumping major version and adding a small tutorial with something like:
should fix it, right? |
I'm not comfortable with this change without getting buy-in from some of the other maintainers. |
@calonso did you run into an issue with latency when changing the namespace. We didnt see your commit before and modified the namespace from scratch. After that, there is a 5-6 second pause between every query. Its possible it could be because we are trying to hit C* 2.1 with it, but that shouldnt matter. Any ideas? |
@46labs Not that I'm aware of, but I remember having a 10 seconds delay on every connection because a removed node hadn't been removed from the peers table of some of the other nodes. I know it has nothing to do with this, but just trying to give ideas. BTW, we discovered that by enabling and reading logging. Maybe if you enable logging you may find out the clue. |
Have you ever attempted to use the driver on C* 2.1? TREVOR FRANCIS [email protected] mailto:[email protected]
|
No, I haven't our cluster is 1.2 |
This PR simply renames the Cassandra symbol into CassandraOld so that this gem can coexist with the newest Cassandra official one from DataStax in the same project.