search

thibaultcha / lua-cassandra

Pure Lua driver for Apache Cassandra
https://thibaultcha.github.io/lua-cassandra
Other
98 stars 35 forks source link

Default Cassandra Read Body Error Message #108

Closed msindwan closed 6 years ago

msindwan commented 6 years ago

We recently encountered an error where the translation was not found for a particular cassandra error. We speculated that this was a write failure and subsequently added it to the list of error codes as per the cassandra protocol. In addition, we added a safe guard for error translations to gracefully handle unsupported error codes.

thibaultcha commented 6 years ago

Oh nice! It seems like WRITE_FAILURE was added in binary protocol v4 (https://github.com/apache/cassandra/blob/trunk/doc/native_protocol_v4.spec#L1106) - while this library was originally developed against the v3. Thanks for the catch!

I am currently in the process of improving the CQL implementation quite a lot (before improving the high-level interfaces). I believe we are having some issues with CI due to third-party dependencies, and those will have to be resolved before I can merge this. Will try to do so as soon as possible!

Thanks again!

thibaultcha commented 6 years ago

Merged, thank you!