[jira] [Comment Edited] (TAJO-437) Timestamp literal support

["DaeMyung Kang (JIRA)" <ji...@apache.org>]

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

DaeMyung Kang edited comment on TAJO-437 at 12/22/13 8:30 AM:
--------------------------------------------------------------

[~hyunsik] and second, serialize function in tajo-common/src/main/java/org/apache/tajo/json/DatumAdapter.java

I think you should add break statement after 

case DATE:
jsonObj.addProperty("value", src.asInt4());
break; <--- need to add this;

default:
    jsonObj.add("body", context.serialize(src));
    break; <-- add break even though it works same.


was (Author: charsyam):
[~hyunsik] and second, serialize function in tajo-common/src/main/java/org/apache/tajo/json/DatumAdapter.java

I think you should add break statement after 

case DATE:
jsonObj.addProperty("value", src.asInt4());
break; <--- need to add this;


> Timestamp literal support
> -------------------------
>
>                 Key: TAJO-437
>                 URL: https://issues.apache.org/jira/browse/TAJO-437
>             Project: Tajo
>          Issue Type: Sub-task
>          Components: operator/expression, parser
>            Reporter: Hyunsik Choi
>            Assignee: Hyunsik Choi
>            Priority: Minor
>              Labels: newbie
>             Fix For: 0.8-incubating
>
>         Attachments: TAJO-437.patch
>
>
> Timestamp literal represents timestamp constant. It has the following bnf grammar.
> {code}
> <timestamp literal>    ::=   TIMESTAMP <timestamp string>
> <timestamp string>    ::=   <quote> <unquoted timestamp string> <quote>
> <unquoted timestamp string>    ::=   <unquoted date string> <space> <unquoted time string>
> <unquoted date string>    ::=   <date value>
> <unquoted time string>    ::=   <time value> [ <time zone interval> ]
> <date value>    ::=   <years value> <minus sign> <months value> <minus sign> <days value>
> <time value>    ::=   <hours value> <colon> <minutes value> <colon> <seconds value>
> <time zone interval>    ::=   <sign> <hours value> <colon> <minutes value>
> {code}



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)