i050510 commented on issue #62: [Olingo 1408] Java 8 DateTime API support by 
Olingo
URL: https://github.com/apache/olingo-odata4/pull/62#issuecomment-558446989
 
 
   I will fix it.
   
   Thanks,
   Ramya
   
   From: mibo <[email protected]>
   Sent: Tuesday, November 26, 2019 2:11 AM
   To: apache/olingo-odata4 <[email protected]>
   Cc: Vasanth, Ramya <[email protected]>; Mention 
<[email protected]>
   Subject: [apache/olingo-odata4] [Olingo 1408] Java 8 DateTime API support by 
Olingo (#62)
   
   
   As the initial #57<https://github.com/apache/olingo-odata4/pull/57> was 
approved I created this branch to make the code formatting and merge easier 
because I have not expected any issue.
   However some test failed which I fixed with commit ec603917e but still one 
test fails (see 
org.apache.olingo.server.tecsvc.data.DataProviderTest.esAllKeyEntity(DataProviderTest.java:71)).
   
   Can you Adrian @agoerler<https://github.com/agoerler> or you Ramya 
@i050510<https://github.com/i050510> please have a look into as you both worked 
on the PR before.
   
   Thanks in advance, Michael
   
   ________________________________
   You can view, comment on, or merge this pull request online at:
   
     https://github.com/apache/olingo-odata4/pull/62
   
   Commit Summary
   
     *   [OLINGO-1408] Support new date time API (#57)
     *   [OLINGO-1408] Applied code formatter
     *   [OLINGO-1408] Minor change to fix tests
   
   File Changes
   
     *   M 
lib/client-core/src/main/java/org/apache/olingo/client/core/communication/request/AsyncRequestWrapperImpl.java<https://github.com/apache/olingo-odata4/pull/62/files#diff-0>
 (2)
     *   M 
lib/commons-core/src/main/java/org/apache/olingo/commons/core/edm/primitivetype/EdmDate.java<https://github.com/apache/olingo-odata4/pull/62/files#diff-1>
 (97)
     *   M 
lib/commons-core/src/main/java/org/apache/olingo/commons/core/edm/primitivetype/EdmDateTimeOffset.java<https://github.com/apache/olingo-odata4/pull/62/files#diff-2>
 (301)
     *   M 
lib/commons-core/src/main/java/org/apache/olingo/commons/core/edm/primitivetype/EdmTimeOfDay.java<https://github.com/apache/olingo-odata4/pull/62/files#diff-3>
 (126)
     *   M 
lib/commons-core/src/test/java/org/apache/olingo/commons/core/edm/primitivetype/EdmDateTest.java<https://github.com/apache/olingo-odata4/pull/62/files#diff-4>
 (41)
     *   M 
lib/commons-core/src/test/java/org/apache/olingo/commons/core/edm/primitivetype/EdmDateTimeOffsetTest.java<https://github.com/apache/olingo-odata4/pull/62/files#diff-5>
 (200)
     *   M 
lib/commons-core/src/test/java/org/apache/olingo/commons/core/edm/primitivetype/EdmTimeOfDayTest.java<https://github.com/apache/olingo-odata4/pull/62/files#diff-6>
 (96)
     *   M 
lib/commons-core/src/test/java/org/apache/olingo/commons/core/edm/primitivetype/PrimitiveTypeBaseTest.java<https://github.com/apache/olingo-odata4/pull/62/files#diff-7>
 (31)
     *   M 
lib/server-test/src/test/java/org/apache/olingo/server/core/serializer/json/ODataJsonSerializerTest.java<https://github.com/apache/olingo-odata4/pull/62/files#diff-8>
 (38)
   
   Patch Links:
   
     *   https://github.com/apache/olingo-odata4/pull/62.patch
     *   https://github.com/apache/olingo-odata4/pull/62.diff
   
   —
   You are receiving this because you were mentioned.
   Reply to this email directly, view it on 
GitHub<https://github.com/apache/olingo-odata4/pull/62?email_source=notifications&email_token=ANWF5CZQ5X35SRXNIRYDYZDQVQZ5BA5CNFSM4JROPFWKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4H35TESA>,
 or 
unsubscribe<https://github.com/notifications/unsubscribe-auth/ANWF5C5YBT2TVFHXYSGH4ULQVQZ5DANCNFSM4JROPFWA>.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to