[jira] [Commented] (TINKERPOP-2264) Gremlin Python should deserialize g:Date to UTC

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

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

ASF GitHub Bot commented on TINKERPOP-2264:
-------------------------------------------

humbleengineer opened a new pull request #1293:
URL: https://github.com/apache/tinkerpop/pull/1293


   The datetime conversion tests didn't pass when running on a Mac in London, so they've been updated to use `calendar.timegm` just like the code under test was in the [original PR for TINKERPOP-2264](https://github.com/apache/tinkerpop/pull/1165).
   
   Also, in order to run & debug the integration tests I updated gremlin-python's pom.xml to refer to a `python` executable rather than `python2`.  Since [Docker on Mac runs in its own VM](https://docs.docker.com/docker-for-mac/docker-toolbox/#the-docker-desktop-on-mac-environment) I updated the integration test `gremlin-server.sh` to explicitly publish its ports using `-p`.  Happy to revert these if they're too environment-specific.
   
   `gremlin-python` install target builds clean using IntelliJ Maven support (I get a javadoc-related error building from command line).


----------------------------------------------------------------
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:
users@infra.apache.org


> Gremlin Python should deserialize g:Date to UTC
> -----------------------------------------------
>
>                 Key: TINKERPOP-2264
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-2264
>             Project: TinkerPop
>          Issue Type: Bug
>          Components: python
>    Affects Versions: 3.3.7
>            Reporter: Stephen Mallette
>            Assignee: Stephen Mallette
>            Priority: Critical
>             Fix For: 3.3.8, 3.4.3, 3.5.0
>
>
> {{g:Date}} seems to be written assuming UTC by Jackson, but Python deserializes with local timezone. First mentioned here:
> https://groups.google.com/d/msg/gremlin-users/KqBgsVhXtjs/NvGUrDxEAQAJ



--
This message was sent by Atlassian Jira
(v8.3.4#803005)