[
https://issues.apache.org/jira/browse/TINKERPOP-2206?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16831581#comment-16831581
]
ASF GitHub Bot commented on TINKERPOP-2206:
-------------------------------------------
spmallette commented on pull request #1107: TINKERPOP-2206 Fixed g:List
serialization
URL: https://github.com/apache/tinkerpop/pull/1107
https://issues.apache.org/jira/browse/TINKERPOP-2206
Seems like javascript arrays were being serialized as naked JSON objects
rather than g:List. Java serialization seemed to forgive that for some reason,
but prevented certain operations to really work right in the process like
P.within() logic from TINKERPOP-2199 which was also fixed in this change.
There may be nicer ways to correct these issues from a javascript
programming perspective - I just kinda hammered something in there. Happy to
hear better ways to do things.
Builds with `mvn clean install`:
VOTE +1
----------------------------------------------------------------
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]
> Certain types in javascript don't appear to serialize with a GraphSON type
> --------------------------------------------------------------------------
>
> Key: TINKERPOP-2206
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2206
> Project: TinkerPop
> Issue Type: Bug
> Components: javascript
> Affects Versions: 3.3.6
> Reporter: stephen mallette
> Priority: Major
>
> While fixing TINKERPOP-2199 I noticed that gremlin-javascript had a similar
> issue with {{P}} but then upon further investigation realized that it seemed
> like arrays were not being properly written to {{g:List}} - it was just being
> written as a naked JSON collection. It seems like the Java deserializer is a
> bit forgiving in this respect.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
