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

Remove support for java serialized Traversal

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Done
    • 3.2.1
    • 3.2.2
    • server

    Description

      Given TINKERPOP-1278 and Bytecode serialization of a Traversal the old method of serializing Traversal with java serialization isn't that useful. There seems to be little point in deprecating that functionality because the only library that supports that protocol is gremlin-driver which will now use the new approach. Dropping it completely wouldn't break anyone's code though we still should consider it a "breaking" change.

      Attachments

        Issue Links

          Activity

            Ramzi Ramzi Oueslati added a comment -

            Hi

            What if we use JMeter to run tests against Gremlin Server (JMeter does not use gremlin driver) ?

            Ramzi Ramzi Oueslati added a comment - Hi What if we use JMeter to run tests against Gremlin Server (JMeter does not use gremlin driver) ?

            I don't follow. How are you using JMeter to send traversals to Gremlin Server if you don't use the java driver?

            spmallette Stephen Mallette added a comment - I don't follow. How are you using JMeter to send traversals to Gremlin Server if you don't use the java driver?
            Ramzi Ramzi Oueslati added a comment -

            We use a JMeter websocket plugin and send gremlin requests.
            Here is a example of JSON sent through the websocket :

            { "requestId":"1d6d02bd-8e56-421d-9438-3bd6d0079ff1",
            "op":"eval",
            "processor":"",
            "args":{"gremlin":"g.V(someNode)",
            "bindings":

            {"someNode":"someValue"}

            ,
            "language":"gremlin-groovy"}
            }

            Ramzi Ramzi Oueslati added a comment - We use a JMeter websocket plugin and send gremlin requests. Here is a example of JSON sent through the websocket : { "requestId":"1d6d02bd-8e56-421d-9438-3bd6d0079ff1", "op":"eval", "processor":"", "args":{"gremlin":"g.V(someNode)", "bindings": {"someNode":"someValue"} , "language":"gremlin-groovy"} }

            that's gremlin-groovy - that hasn't changed. this ticket was about getting rid of serialization of a Traversal via java object serialization. you're not using that.

            spmallette Stephen Mallette added a comment - that's gremlin-groovy - that hasn't changed. this ticket was about getting rid of serialization of a Traversal via java object serialization. you're not using that.
            Ramzi Ramzi Oueslati added a comment -

            Sorry Stephen. My mistake.
            I upgraded to 3.2.2 and had a few problems and I though it was related.

            Ramzi Ramzi Oueslati added a comment - Sorry Stephen. My mistake. I upgraded to 3.2.2 and had a few problems and I though it was related.

            People

              spmallette Stephen Mallette
              spmallette Stephen Mallette
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: