cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Benjamin Lerer (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CASSANDRA-12850) Add the duration type to the protocol V5
Date Fri, 20 Jan 2017 12:49:26 GMT

    [ https://issues.apache.org/jira/browse/CASSANDRA-12850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15831685#comment-15831685
] 

Benjamin Lerer commented on CASSANDRA-12850:
--------------------------------------------

||[3.11|https://github.com/apache/cassandra/compare/trunk...blerer:12850-3.11]|[utest|http://cassci.datastax.com/view/Dev/view/blerer/job/blerer-12850-3.11-testall/]|[dtest|http://cassci.datastax.com/view/Dev/view/blerer/job/blerer-12850-3.11-dtest/]|
||[trunk|https://github.com/apache/cassandra/compare/trunk...blerer:12850-trunk]|[utest|http://cassci.datastax.com/view/Dev/view/blerer/job/blerer-12850-trunk-testall/]|[dtest|http://cassci.datastax.com/view/Dev/view/blerer/job/blerer-12850-trunk-dtest/]|

The patch add {{Duration}} to the protocol specification and to the {{DataType}}.
While running the tests, I faced an {{IllegalStateException}} in {{OptionCodec}} caused by
the fact that 2 {{DataType}}s where having the same id (the one of {{CUSTOM}}.
As {{OptionCodec}} was only used by {{DataType}} I simplified it and converted it into an
inner class. That change helped to fix the problem as the {{protocolVersion}} of the {{DataType}}
was needed. 




> Add the duration type to the protocol V5
> ----------------------------------------
>
>                 Key: CASSANDRA-12850
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-12850
>             Project: Cassandra
>          Issue Type: Sub-task
>          Components: CQL
>            Reporter: Benjamin Lerer
>            Assignee: Benjamin Lerer
>             Fix For: 3.x
>
>
> The Duration type need to be added to the protocol specifications.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message