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

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

spmallette commented on a change in pull request #1293:
URL: https://github.com/apache/tinkerpop/pull/1293#discussion_r438046045



##########
File path: docker/gremlin-server.sh
##########
@@ -57,7 +57,7 @@ echo "Using Gremlin Server $GREMLIN_SERVER_VERSION"
 sed -e "s/GREMLIN_SERVER_VERSION\$/${GREMLIN_SERVER_VERSION}/" 
docker/gremlin-server/Dockerfile.template > Dockerfile
 
 docker build -t tinkerpop:${BUILD_TAG} .
-docker run ${TINKERPOP_TEST_DOCKER_OPTS} ${REMOVE_CONTAINER} -ti -v 
"${HOME}"/.groovy:/root/.groovy -v "${HOME}"/.m2:/root/.m2 
tinkerpop:${BUILD_TAG} ${@}
+docker run ${TINKERPOP_TEST_DOCKER_OPTS} ${REMOVE_CONTAINER} -ti -p 
45940:45940 -p 45941:45941 -v "${HOME}"/.groovy:/root/.groovy -v 
"${HOME}"/.m2:/root/.m2 tinkerpop:${BUILD_TAG} ${@}

Review comment:
       I'm still not smart enough about docker to know why this worked all this 
time without the `-p` - @robertdale or @dkuppitz  any ideas?




----------------------------------------------------------------
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]


> 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)

Reply via email to