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

Install plugin always fails after first unresolved dependency

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.2-incubating
    • Fix Version/s: 3.1.3, 3.2.1
    • Component/s: groovy
    • Labels:
      None

      Description

      Original issue: The DependencyGrabberTest currently require ordered execution based on the name of the test. In some environments this seems to cause the tests to fail (as they may execute out of order). Generally speaking though it seems it would be best if the tests didn't have dependencies on each other in order to pass.

      Actual issue: It turns out that there is a bug in Groovy Grapes. When there is a failed call to Grape.resolve(), GrapeIvy.resolve() hangs onto a reference to the IvyGrabRecord for the failed dependency. When subsequent resolve() calls are made, those failed records are still there, so it will continue to fail from that point on. This explains why the ordering was important in the TinkerPop test case.

               \,,,/
               (o o)
      -----oOOo-(3)-oOOo-----
      plugin activated: tinkerpop.server
      plugin activated: tinkerpop.utilities
      plugin activated: tinkerpop.tinkergraph
      gremlin> :install bogus bogus 1.0
      ==>Error grabbing Grapes -- [unresolved dependency: bogus#bogus;1.0: not found]
      gremlin> :install org.apache.tinkerpop hadoop-gremlin 3.0.1-incubating
      ==>Error grabbing Grapes -- [unresolved dependency: bogus#bogus;1.0: not found]
      

      When Groovy 2.4.6 ships with the fix for GROOVY-7649, the @Ignore in the test case can be removed if TinkerPop bumps up to the new Groovy version.

        Attachments

          Activity

            People

            • Assignee:
              pluradj Jason Plurad
              Reporter:
              spmallette Stephen Mallette
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: