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

Cheolsoo Park commented on PIG-3014:
------------------------------------

Thanks Rohini.

In fact, I asked that question on the dev mailing list a while ago:
http://search-hadoop.com/m/OVyoR1Ktpcy/Adding+new+test+cases+to+TestBuiltin.java&subj=Adding+new+test+cases+to+TestBuiltin+java

Julien said that each built-in UDF should have its own test suite, so I 
followed it in PIG-2881. I guess that the same applies to CurrentTime().

Please anyone correct me if I am wrong.
                
> CurrentTime() UDF has undesirable characteristics
> -------------------------------------------------
>
>                 Key: PIG-3014
>                 URL: https://issues.apache.org/jira/browse/PIG-3014
>             Project: Pig
>          Issue Type: Bug
>            Reporter: Jonathan Coveney
>            Assignee: Jonathan Coveney
>             Fix For: 0.12
>
>         Attachments: PIG-3014-0.patch, PIG-3014-1.patch, PIG-3014-2.patch, 
> PIG-3014-3.patch
>
>
> As part of the explanation of the new DateTime datatype I noticed that we had 
> added a CurrentTime() UDF. The issue with this UDF is that it returns the 
> current time _of every exec invocation_, which can lead to confusing results. 
> In PIG-1431 I proposed a way such that every instance of the same NOW() will 
> return the same time, which I think is better. Would enjoy thoughts.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to