Minor Description
We long ago stopped espousing the use of the groovy benchmarks/performance libs in Gremlin Console. Not sure why they are still packaged in there with the distribution except that the utilities plugin references the classes in its auto-import. I also think they are somewhat bound to Groovy 2.4.x and TinkerPop 3.4.x is on Groovy 2.5.x. I'd say it's best to remove them from both tp33 and master at this point.
Marking this with the "breaking" label because it means uses who might oddly depend on gremlin-console for this dependency will need to add it themselves now and not rely on it transitively. Highly unlikely I would imagine. It is also breaking if folks somehow actually use these classes in their use of the console.The workaround to bring these libs back in is pretty simple - just use the :install command and do some imports. This can be documented in the upgrade docs. Taken all together, this seems like a low risk breaking change.
Attachments
Issue Links
- links to