[TINKERPOP-2562] Remove GraphSON 2 option in TraversalOpProcessor

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Done
• Affects Version/s: 3.4.11
• Fix Version/s: 3.6.0
• Component/s: server
• Labels: breaking
• Assignee: Stephen Mallette
• Reporter: Stephen Mallette
• Created: 11/May/21 12:00
• Updated: 01/Jun/21 16:54
• Resolved: 01/Jun/21 16:54

Description

There is an option for the TraversalOpProcessor to take a GraphSON 2 string rather than a Bytecode object. Perhaps that is the way it is for legacy reasons - not sure, but being bound to GraphSON 2 seems odd. Need to investigate further, but I would think that we just want to expect a Bytecode object and that's it:

https://github.com/apache/tinkerpop/blob/3.4.11/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/op/traversal/TraversalOpProcessor.java#L82