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

Zhijie Shen updated PIG-1314:
-----------------------------

    Attachment: PIG-1314-5.patch

Hi Thejas,

I attached my newest patch (the same as that in my previous email sent to you). 
Compared the last version, there's following improvement:

1. More test cases have been added, such that the test cases are nearly 
completed.

2. Fix some bugs according to the test cases, including the builtin functions 
(e.g., argToFuncMapping).

3. I add some more builtin functions: MilliSecondsBetween, GetMilliSecond, 
ToMillSeconds, since the granularity of pig DateTime is set to millsecond.

I've also some comments:

1. DiffDate behaves similar to DaysBetween, except that the former function 
return opposite values if two arguments change their order.

2. According to your last response, I'm not clear how the default timezone of 
client can be sent to the server with the code. In my opinion, the default 
timezone should be specified on the server side by configuration, which should 
be taken care of by administrators. How do you think about this.

I think this patch is close to commit. Please check it out. Thanks!
                
> Add DateTime Support to Pig
> ---------------------------
>
>                 Key: PIG-1314
>                 URL: https://issues.apache.org/jira/browse/PIG-1314
>             Project: Pig
>          Issue Type: Bug
>          Components: data
>    Affects Versions: 0.7.0
>            Reporter: Russell Jurney
>            Assignee: Zhijie Shen
>              Labels: gsoc2012
>         Attachments: joda_vs_builtin.zip, PIG-1314-1.patch, PIG-1314-2.patch, 
> PIG-1314-3.patch, PIG-1314-4.patch, PIG-1314-5.patch
>
>   Original Estimate: 672h
>  Remaining Estimate: 672h
>
> Hadoop/Pig are primarily used to parse log data, and most logs have a 
> timestamp component.  Therefore Pig should support dates as a primitive.
> Can someone familiar with adding types to pig comment on how hard this is?  
> We're looking at doing this, rather than use UDFs.  Is this a patch that 
> would be accepted?
> This is a candidate project for Google summer of code 2012. More information 
> about the program can be found at 
> https://cwiki.apache.org/confluence/display/PIG/GSoc2012

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to