3

Working with py2neo and I'm getting the error below when trying to append a transaction:

statement ="MERGE (a:Person {name:\""+actorName+"\"}) "\
            "\n"\
            "MERGE (b:Series {title:\""+actorsFields[3]+"\", year:\""+actorsFields[5]+"\"}) "\
            "\n"\
            "CREATE UNIQUE (a)-[:ACTED_IN]->(b)"\
            "RETURN a,b"
print(statement)
tx.append(statement)

The traceback is:

Traceback (most recent call last):
  File "/Volumes/PyCharm CE/PyCharm CE.app/Contents/helpers/pydev/pydevd.py", line 2222, in <module>
    globals = debugger.run(setup['file'], None, None)
  File "/Volumes/PyCharm CE/PyCharm CE.app/Contents/helpers/pydev/pydevd.py", line 1648, in run
    pydev_imports.execfile(file, globals, locals)  # execute the script
  File "/Users/Thibault/PycharmProjects/movieGraph/src/mainCypher.py", line 110, in <module>
    tx.append(statement)
  File "/Library/Python/2.7/site-packages/py2neo/cypher/core.py", line 220, in append
    self.__assert_unfinished()
  File "/Library/Python/2.7/site-packages/py2neo/cypher/core.py", line 192, in __assert_unfinished
    raise Finished(self)
py2neo.error.Finished

any ideas?

El Bert
  • 2,958
  • 1
  • 28
  • 36
Thibault Lefevre
  • 83
  • 4

1 Answers1

2

You will get this error if you call tx.commit() twice without a tx = graph.cypher.begin() in between. This is an easy mistake to make if you are trying to chunk your commits. To be more explicit:

 #This will give the above error
 tx = graph.cypher.begin()
 for i in range(0,10):
      tx.append(statement="foo",parameters=bar)
      tx.commit()

 #This will work fine
 for i in range(0,10):
      tx = graph.cypher.begin()
      tx.append(statement="foo",parameters=bar)
      tx.commit()
sjc
  • 1,117
  • 3
  • 19
  • 28
  • Although the syntax changed a bit, the need to begin the transaction after each commit is still needed in v4. You'd think they'd include an important detail like this in the documentation in an obvious way. – Aaron Bramson Nov 27 '18 at 05:57