cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stefania (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-11274) cqlsh: interpret CQL type for formatting blob types
Date Wed, 02 Mar 2016 01:49:18 GMT


Stefania commented on CASSANDRA-11274:

bq. I noted: inet is now being formatted green instead of yellow. It actually looks like it
wasn't hitting {{@formatter_for('inet')}} in previously, and now it is. I guess this is probably
correct, but I'm just pointing out the difference.

Thanks for pointing this out, I hadn't noticed it. Green is the correct color for inet, the
colors are specified at the bottom of _displaying.py_.

bq. Without Cython, the formatter assumptions fail:

I fixed the formatter thanks. I've also fixed the COPY FROM date converter.

bq. With Cython, the deserializer has the same overflow that the patch is working around:

For the time being I've removed the cython deserializer so that we fall back to the patched
function. I will also apply this fix to the 3.0 patch of CASSANDRA-11053 since CASSANDRA-10625
was delivered to 3.0 as well.

Latest changes are [here|].

CI restarted.

> cqlsh: interpret CQL type for formatting blob types
> ---------------------------------------------------
>                 Key: CASSANDRA-11274
>                 URL:
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Tools
>            Reporter: Stefania
>            Assignee: Stefania
>             Fix For: 3.x
> During the development of CASSANDRA-11053 we have added changes to the cqlsh formatting
code so that we can format {{blob}} types correctly even if they are represented as {{str}}
rather than {{bytearray}}.
> At the moment we ensure {{blob}} are of type {{bytearray}} via the following shortcut:
> {code}
> cassandra.cqltypes.BytesType.deserialize = staticmethod(lambda byts, protocol_version:
> {code}
> After CASSANDRA-11053 is committed there will be a similar shortcut to override the fast
serializers implemented in cython. 
> Decoding the CQL type is safer in that it decouples cqlsh formatting from the types returned
by the driver deserializers but it is also unnecessary for CASSANDRA-11053 performance goals
and risky for older releases. 
> Therefore this ticket delivers this functionality but only on trunk.

This message was sent by Atlassian JIRA

View raw message