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

James Taylor edited comment on PHOENIX-1765 at 4/11/15 5:25 AM:
----------------------------------------------------------------

Thanks, [~ayingshu]. Couple of minor things:
- Always add built-in functions to the end of the ExpressionType enum. Don't 
worry about them not being grouped with the other date/time functions
- Upsert a few more rows to make sure every row isn't being matched.



was (Author: jamestaylor):
Thanks, [~ayingshu]. Couple of minor things:
- Always add built-in functions to the end of the ExpressionType enum. Don't 
worry about them not being grouped with the other date/time functions
- Upsert a few more values to make sure every row isn't being matched.


> Add DAYOFMONTH built-in function
> --------------------------------
>
>                 Key: PHOENIX-1765
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-1765
>             Project: Phoenix
>          Issue Type: Bug
>            Reporter: Alicia Ying Shu
>            Assignee: Alicia Ying Shu
>              Labels: Newbie
>         Attachments: Phoenix-1765.patch
>
>
> Implement DAYOFMONTH(date/time/timestamp) built-in function: An integer from 
> 1 to 31 representing the day of the month in date.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to