Uploaded image for project: 'TinkerPop'
  1. TinkerPop
  2. TINKERPOP-1005

gremlin-server should throw a 500 error on commit() failure

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Won't Do
    • Affects Version/s: 3.0.2-incubating
    • Fix Version/s: None
    • Component/s: server
    • Labels:
      None

      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.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                dmill Dylan Millikin
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: