Mitigate cryptic "cannot parse as hex bytes" error via explicit cqlVersion test
#109
Conversation
|
So, there are just a couple of concerns I have. First is that this will break backward compatibility as Helenus currently defaults to CQL3. (see: connections.js) for C* 1.2 and CQL 2 for lesser versions. The desired behavior is that you do not need to explicitly set the version, which makes me think there is a bug in the CQL selection. If you look [here[(https://github.com/simplereach/helenus/blob/master/lib/connection.js#L452-L470) you will see that the default version is 2.0.0 if the CQL version has not been set to 3. CQL version 3 is only allowed after is has been set OR if the version is greater than 19.33.0 (C* 1.2.0). That is where I think the issue is. Can you verify if changing this line fixes the issue? it should look something like: |
… to be 'or' instead of 'and'
|
What you described fixes the issue. |
Mitigate cryptic "cannot parse as hex bytes" error via explicit `cqlVersion` test. closes @108
This should mitigate errors and confusion of the type described in #108.