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

gremlin.sh -e doesn't log line numbers for errors

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Implemented
    • Affects Version/s: 3.1.1-incubating
    • Fix Version/s: 3.2.1
    • Component/s: console
    • Labels:
      None

      Description

      Execute a script containing the following code

      import org.apache.tinkerpop.gremlin.tinkergraph.structure.*
      graph = TinkerGraph.open()
      graph.noexist()
      

      The error is as follows with no line number from the script.

      $ ./gremlin.sh -e test.groovy 
      javax.script.ScriptException: groovy.lang.MissingMethodException: No signature of method: org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerGraph.noexist() is applicable for argument types: () values: []
      Possible solutions: notify(), wait(), collect()
      

      It would be really nice to get a line number from the script for the error message.

      Some bugs do generate line numbers. Those seem to be certain kinds of syntax exrrors.

      import org.apache.tinkerpop.gremlin.tinkergraph.structure.*
      graph = TinkerGraph.open()
      graph.noexist(
      

      Output

      $ ./gremlin.sh -e test.groovy 
      org.codehaus.groovy.control.MultipleCompilationErrorsException: startup failed:
      Script1.groovy: 4: unexpected token:  @ line 4, column 1.
      1 error
      

        Issue Links

          Activity

          Hide
          spmallette stephen mallette added a comment -

          Implemented on TINKERPOP-1268

          Show
          spmallette stephen mallette added a comment - Implemented on TINKERPOP-1268
          Hide
          spmallette stephen mallette added a comment -

          This is being implemented on TINKERPOP-1268

          Show
          spmallette stephen mallette added a comment - This is being implemented on TINKERPOP-1268

            People

            • Assignee:
              spmallette stephen mallette
              Reporter:
              aweisberg Ariel Weisberg
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development