neo4j-apoc-procedures icon indicating copy to clipboard operation
neo4j-apoc-procedures copied to clipboard

Deprecate mapParallel procedure and update docs

Open conker84 opened this issue 2 years ago • 3 comments
trafficstars

Deprecate the procs and update the docs by explaining how to move to the Parallel runtime

conker84 avatar Nov 24 '23 12:11 conker84

Please do not delete apoc.cypher.mapParallel2 because at the moment, the new PARALLEL runtime cannot fully replace it. Details https://stackoverflow.com/questions/77509091/neo4j-parallel-runtime-query-returns-different-result

Example: I have many scenarios where I currently use apoc.cypher.mapParallel2 to read data in the same transaction where I'm updating it. I would like to transition to the PARALLEL runtime, but I am unable to do so due to this limitation.

Also, I'm testing the new Neo4j PARALLEL runtime and am unable to pass my tests because my queries return different results (different elements order) compared to SLOTTED or PIPELINED.

alexanoid avatar Jan 13 '24 23:01 alexanoid

For example I'm unable to port my query which is based right now on apoc.cypher.mapParallel2 to PARALLEL runtime https://stackoverflow.com/questions/77814894/neo4j-on-parallel-runtime-returns-different-results-different-elements-order-c

alexanoid avatar Jan 14 '24 13:01 alexanoid

Additionally, with the help of apoc.cypher.mapParallel2, I can parallelize only a specific part of the query. This is a feature which is not available with the PARALLEL runtime.

With apoc.cypher.mapParallel2 I may use chaining with parallel and non-parallel executions in one single query. Please keep this procedure in the library and leave us a choice where to use PARALLEL runtime and where apoc.cypher.mapParallel2.

alexanoid avatar Jan 15 '24 22:01 alexanoid