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

Christian Amend edited comment on OLINGO-793 at 10/8/15 10:52 AM:
------------------------------------------------------------------

Hi Ramesh,

please be aware that although the precision is optional for temporal types the 
default is zero : ". If no value is specified, the temporal property has a 
precision of zero." 
(http://docs.oasis-open.org/odata/odata/v4.0/errata02/os/complete/part3-csdl/odata-v4.0-errata02-os-part3-csdl-complete.html#_Toc406397957)

So the expcetion here is correct as the code is only executed for fractional 
seconds bigger than 0. 

If this answer satisfies you please close the issue. In case you would like to 
adjust the error message lets discuss :)

Best Regards,
Christian


was (Author: chrisam):
Hi Ramesh,

please be aware that although the precision is optional for temporal types the 
default is zero : ". If no value is specified, the temporal property has a 
precision of zero." 
(http://docs.oasis-open.org/odata/odata/v4.0/errata02/os/complete/part3-csdl/odata-v4.0-errata02-os-part3-csdl-complete.html#_Toc406397957)

So the expcetion here is correct as the code is only executed for fractional 
seconds bigger than 0. 

Best Regards,
Christian

> Precision on EdmDateTimeOffset should be optional
> -------------------------------------------------
>
>                 Key: OLINGO-793
>                 URL: https://issues.apache.org/jira/browse/OLINGO-793
>             Project: Olingo
>          Issue Type: Bug
>          Components: odata4-commons
>    Affects Versions: (Java) V4 4.0.0
>            Reporter: Ramesh Reddy
>
> According to [1] the precision attribute on EdmdateTimeOffset is optional 
> facet that can be defined. However the code [2] fails with 
> IllegalArgumentException is it found to be null.
> [1] 
> http://docs.oasis-open.org/odata/odata/v4.0/errata02/os/complete/part3-csdl/odata-v4.0-errata02-os-part3-csdl-complete.html#_Toc406397957
> [2] 
> https://github.com/apache/olingo-odata4/blob/master/lib/commons-core/src/main/java/org/apache/olingo/commons/core/edm/primitivetype/EdmDateTimeOffset.java#L253



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

Reply via email to