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

Remove gperfutils from Gremlin Console

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Done
    • Affects Version/s: 3.3.5
    • Fix Version/s: 3.3.7, 3.4.2
    • Component/s: console
    • Labels:

      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

            Activity

              People

              • Assignee:
                spmallette stephen mallette
                Reporter:
                spmallette stephen mallette
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: