You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Jonathan Ellis (JIRA)" <ji...@apache.org> on 2011/06/01 18:30:48 UTC

[jira] [Commented] (CASSANDRA-2530) Additional AbstractType data type definitions to enrich CQL

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

Jonathan Ellis commented on CASSANDRA-2530:
-------------------------------------------

Isn't date throwing away information when it encodes to long (the time zone)?

Also not a fan of copy/pasting the iso patterns, go ahead and reference the copy in TUUIDT, or move those somewhere more generic.

Using compareUnsigned in the boolean compare method seems like overkill.

Otherwise looks fine to me.

> Additional AbstractType data type definitions to enrich CQL
> -----------------------------------------------------------
>
>                 Key: CASSANDRA-2530
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2530
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 0.8.0 beta 2
>            Reporter: Rick Shaw
>            Priority: Trivial
>              Labels: cql
>         Attachments: patch-to-add -5 -new-AbstractTypes-v2.txt, patch-to-add -5 -new-AbstractTypes-v3.txt, patch-to-add -5 -new-AbstractTypes.txt, patch-to-add-4-new-AbstractTypes-and-CQL-support-v4.txt
>
>
> Provide 5 additional Datatypes: ByteType, DateType, BooleanType, FloatType, DoubleType.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira