[jira] [Commented] (TINKERPOP-1799) Failure to serialize path() in gremlin-python

2017-10-20 Thread ASF GitHub Bot (JIRA) 

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

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

Github user asfgit closed the pull request at:

https://github.com/apache/tinkerpop/pull/732


> Failure to serialize path() in gremlin-python
> -
>
> Key: TINKERPOP-1799
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1799
> Project: TinkerPop
>  Issue Type: Bug
>  Components: python
>Affects Versions: 3.3.0
>Reporter: stephen mallette
>Assignee: stephen mallette
>Priority: Critical
> Fix For: 3.3.1
>
>
> {code}
> >>> g.V().hasLabel('person').both().path().toList()
> [[u'@type', u'@value'], [u'@type', u'@value'], [u'@type', u'@value'], 
> [u'@type', u'@value'], [u'@type', u'@value'], [u'@type', u'@value'], 
> [u'@type', u'@value'], [u'@type', u'@value']]
> {code}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (TINKERPOP-1799) Failure to serialize path() in gremlin-python

2017-10-20 Thread ASF GitHub Bot (JIRA) 

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

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

Github user robertdale commented on the issue:

https://github.com/apache/tinkerpop/pull/732
  
VOTE +1



> Failure to serialize path() in gremlin-python
> -
>
> Key: TINKERPOP-1799
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1799
> Project: TinkerPop
>  Issue Type: Bug
>  Components: python
>Affects Versions: 3.3.0
>Reporter: stephen mallette
>Assignee: stephen mallette
>Priority: Critical
> Fix For: 3.3.1
>
>
> {code}
> >>> g.V().hasLabel('person').both().path().toList()
> [[u'@type', u'@value'], [u'@type', u'@value'], [u'@type', u'@value'], 
> [u'@type', u'@value'], [u'@type', u'@value'], [u'@type', u'@value'], 
> [u'@type', u'@value'], [u'@type', u'@value']]
> {code}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (TINKERPOP-1799) Failure to serialize path() in gremlin-python

2017-10-20 Thread ASF GitHub Bot (JIRA) 

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

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

Github user jorgebay commented on the issue:

https://github.com/apache/tinkerpop/pull/732
  
VOTE +1


> Failure to serialize path() in gremlin-python
> -
>
> Key: TINKERPOP-1799
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1799
> Project: TinkerPop
>  Issue Type: Bug
>  Components: python
>Affects Versions: 3.3.0
>Reporter: stephen mallette
>Assignee: stephen mallette
>Priority: Critical
> Fix For: 3.3.1
>
>
> {code}
> >>> g.V().hasLabel('person').both().path().toList()
> [[u'@type', u'@value'], [u'@type', u'@value'], [u'@type', u'@value'], 
> [u'@type', u'@value'], [u'@type', u'@value'], [u'@type', u'@value'], 
> [u'@type', u'@value'], [u'@type', u'@value']]
> {code}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (TINKERPOP-1799) Failure to serialize path() in gremlin-python

2017-10-13 Thread ASF GitHub Bot (JIRA) 

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

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

GitHub user spmallette opened a pull request:

https://github.com/apache/tinkerpop/pull/732

TINKERPOP-1799 Fixed serialization of Path objects for python.

https://issues.apache.org/jira/browse/TINKERPOP-1799

This issue only related to GraphSON 3.0 and was fairly straightforward to 
fix. Tests did not pick up the failure because they continued to use GraphSON 
2.0 format in the test itself.

VOTE +1

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/apache/tinkerpop TINKERPOP-1799

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/tinkerpop/pull/732.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #732


commit d5ed2f7cb9a110453667be0b1dbfc6bf9c0383a6
Author: Stephen Mallette 
Date:   2017-10-13T16:34:42Z

TINKERPOP-1799 Fixed serialization of Path objects for python.

This issue only related to GraphSON 3.0 and was fairly straightforward to 
fix. Tests did not pick up the failure because they continued to use GraphSON 
2.0 format in the test itself.




> Failure to serialize path() in gremlin-python
> -
>
> Key: TINKERPOP-1799
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1799
> Project: TinkerPop
>  Issue Type: Bug
>  Components: python
>Affects Versions: 3.3.0
>Reporter: stephen mallette
>Assignee: stephen mallette
>Priority: Critical
> Fix For: 3.3.1
>
>
> {code}
> >>> g.V().hasLabel('person').both().path().toList()
> [[u'@type', u'@value'], [u'@type', u'@value'], [u'@type', u'@value'], 
> [u'@type', u'@value'], [u'@type', u'@value'], [u'@type', u'@value'], 
> [u'@type', u'@value'], [u'@type', u'@value']]
> {code}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (TINKERPOP-1799) Failure to serialize path() in gremlin-python

2017-10-13 Thread stephen mallette (JIRA) 

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

stephen mallette commented on TINKERPOP-1799:
-

This issue is turning up a bit of a mess. IO docs for GraphSON 3.0 are all 
messed up (which means compatibility tests are off too) and python tests for 
GraphSON 3.0 appear to be off. The These issues were sorted in this commit docs 
now as a commit separate from this issue:

https://github.com/apache/tinkerpop/commit/4bcf8ae5c734f877d0e2d003ec7af3b2fc37e116

Looks big but it's really just code generation stuff which is why i CTR'd it.

> Failure to serialize path() in gremlin-python
> -
>
> Key: TINKERPOP-1799
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1799
> Project: TinkerPop
>  Issue Type: Bug
>  Components: python
>Affects Versions: 3.3.0
>Reporter: stephen mallette
>Assignee: stephen mallette
>Priority: Critical
> Fix For: 3.3.1
>
>
> {code}
> >>> g.V().hasLabel('person').both().path().toList()
> [[u'@type', u'@value'], [u'@type', u'@value'], [u'@type', u'@value'], 
> [u'@type', u'@value'], [u'@type', u'@value'], [u'@type', u'@value'], 
> [u'@type', u'@value'], [u'@type', u'@value']]
> {code}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)