[TINKERPOP-1005] gremlin-server should throw a 500 error on commit() failure

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Won't Do
• Affects Version/s: 3.0.2-incubating
• Component/s: server
• Assignee: Unassigned
• Reporter: Dylan Millikin
• Created: 29/Nov/15 11:59
• Updated: 15/Jun/17 18:20
• Resolved: 15/Jun/17 18:20

Description

Currently gremlin-server throws a 597 - SCRIPT EVALUATION ERROR on commit failures (for concurrency reasons for example). This isn't really the correct error to return in those cases. Drivers would expect a 500 - SERVER ERROR instead for their retry-failure strategies.

Of course this could take on another form and somehow allow drivers to differentiate commit failures from other exceptions. So in any event this is probably going to depend on https://issues.apache.org/jira/browse/TINKERPOP3-1004 yet also require a change to the server code.

Issue Links

• depends upon: TINKERPOP-1004 Make Transaction.commit() failures consistent across implementations.