[jira] [Commented] (TINKERPOP-1811) Improve error reporting for serialization errors between gremlin-python and gremlin-server

2017-11-23 Thread ASF GitHub Bot (JIRA)

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

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

Github user aolieman commented on the issue:

https://github.com/apache/tinkerpop/pull/753
  
Thanks @spmallette , this should really help.


> Improve error reporting for serialization errors between gremlin-python and 
> gremlin-server
> --
>
> Key: TINKERPOP-1811
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1811
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: python, server
>Affects Versions: 3.2.6
>Reporter: Hugo Arts
>Priority: Minor
>
> We ran into an issue while attempting to add a PartitionStrategy to our 
> traversals. This is a small example of a traversal I tried:
> {code:none}
> remote = DriverRemoteConnection('ws://localhost:8182/gremlin', 
> 'DSE_GRAPH_QUICKSTART.g')
> strategy = PartitionStrategy('community_id', '2097442560', 2097442560)
> g = Graph().traversal().withRemote(remote).withStrategies(strategy)
> print(g.V().toList())
> {code}
> The third argument of PartitionStrategy should be a list. However, the 
> traversal strategies in gremlin-python do not do any kind of argument/type 
> checking, and any mistake you make results in the following error from 
> gremlin server:
> {noformat}
> ERROR [gremlin-server-worker-1] 2017-10-25 10:15:53,285 GREMLIN 
> TraversalOpProcessor.java:365 - Could not deserialize the Traversal instance
> {noformat}
> I've omitted the [full 
> traceback|https://gist.github.com/anonymous/2ad6541d0169256afb1a23fbdc376009] 
> for brevity. There is not much indication as to what has gone wrong. 
> gremlin-python gives this error:
> {noformat}
> gremlin_python.driver.protocol.GremlinServerError: 599: None
> {noformat}
> Which is also unhelpful. We were eventually able to figure out what we were 
> doing wrong, but it would be very nice to have either some input validation 
> on the gremlin-python side, or have gremlin-server be more specific about the 
> nature of the deserialisation error (or both).



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


[GitHub] tinkerpop issue #753: TINKERPOP-1811 Fixed bytecode deserialization error me...

2017-11-23 Thread aolieman
Github user aolieman commented on the issue:

https://github.com/apache/tinkerpop/pull/753
  
Thanks @spmallette , this should really help.


---


[jira] [Commented] (TINKERPOP-1827) Gremlin .NET: Test Suite Runner

2017-11-23 Thread ASF GitHub Bot (JIRA)

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

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

Github user jorgebay commented on the issue:

https://github.com/apache/tinkerpop/pull/754
  
I rebased it against `tp32`, all tests pass.

VOTE +1


> Gremlin .NET: Test Suite Runner
> ---
>
> Key: TINKERPOP-1827
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1827
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: dotnet, test-suite
>Affects Versions: 3.2.6
>Reporter: Jorge Bay
>Assignee: Jorge Bay
>
> Provide a way to run the language agnostic test framework from TINKERPOP-1784 
> on the .NET GLV.



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


[GitHub] tinkerpop issue #754: TINKERPOP-1827 Gremlin .NET: Test Suite Runner

2017-11-23 Thread jorgebay
Github user jorgebay commented on the issue:

https://github.com/apache/tinkerpop/pull/754
  
I rebased it against `tp32`, all tests pass.

VOTE +1


---