cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Chander S Pechetty (JIRA)" <>
Subject [jira] [Updated] (CASSANDRA-6973) timestamp data type does ISO 8601 formats with 'Z' as time zone.
Date Wed, 07 May 2014 07:42:18 GMT


Chander S Pechetty updated CASSANDRA-6973:

    Attachment: trunk-6973_unittest.txt

Good catch on missing non-ISO by removing Z.
- Z was only supporting RFC 822  timezone (-0800). 
- the pattern with two X will cover the RFC 822 which takes care of removing Z. patch v2 takes
care of this
- pattern with single X covers this ticket (Z as well as -08)
- the pattern XXX handles -08:00 
- With the one, two and three letter patterns we complete all the 4 timezone designators for
8601 I stated earlier in the ticket
- did have a test program, moved that into a unit test and uploaded as a separate patch.

On another note maybe its a good idea to rename the iso8601Patterns in TimestampSerializer
to something like casTimestampPatterns which cover both iso and non-ISO dates. what do you
think ?

> timestamp data type does ISO 8601 formats with 'Z' as time zone.
> ----------------------------------------------------------------
>                 Key: CASSANDRA-6973
>                 URL:
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Core
>            Reporter: Juho Mäkinen
>            Assignee: Chander S Pechetty
>            Priority: Trivial
>         Attachments: trunk-6973.txt, trunk-6973_unittest.txt, trunk-6973_v2.txt
> The timestamp data type does not support format where time zone is specified with 'Z'
(as in zulu aka. UTC+0 aka +0000 time zone). Example:
> create table foo(ts timestamp primary key);
> insert into foo(ts) values('2014-04-01T20:17:35+0000'); -- this works
> cqlsh:test> insert into foo(ts) values('2014-04-01T20:17:35Z');
> Bad Request: unable to coerce '2014-04-01T20:17:35Z' to a  formatted date (long)
> The example date was copied directly from ISO 8601 Wikipedia page. The standard says
that "If the time is in UTC, add a Z directly after the time without a space. Z is the zone
designator for the zero UTC offset."
> Tested with cqlsh with 2.0.6 version.

This message was sent by Atlassian JIRA

View raw message