Mailing-List: contact commits-help@cassandra.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cassandra.apache.org
Date: Mon, 29 Feb 2016 05:47:18 +0000 (UTC)
From: "Stefania (JIRA)" <jira@apache.org>
To: commits@cassandra.apache.org
Message-ID: <JIRA.12945318.1456708147000.169766.1456724838052@Atlassian.JIRA>
In-Reply-To: <JIRA.12945318.1456708147000@Atlassian.JIRA>
References: <JIRA.12945318.1456708147000@Atlassian.JIRA>
 <JIRA.12945318.1456708147356@arcas>
Subject: [jira] [Updated] (CASSANDRA-11274) cqlsh: interpret CQL type for
 formatting blob types
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


     [ https://issues.apache.org/jira/browse/CASSANDRA-11274?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Stefania updated CASSANDRA-11274:
---------------------------------
    Description: 
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: bytearray(byts))
{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.


  was:
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: bytearray(byts))
{code}

After CASSANDRA-11053 is committed there will be a similar shortcut to override the fast serializers implemented in cython. This trick was not known initially, which is why the code to interpret CQL types was written.

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.


> cqlsh: interpret CQL type for formatting blob types
> ---------------------------------------------------
>
>                 Key: CASSANDRA-11274
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-11274
>             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: bytearray(byts))
> {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
(v6.3.4#6332)