neo4j / docker-neo4j

Docker Images for the Neo4j Graph Database
Apache License 2.0
331 stars 172 forks source link

5.x load test uses the new load command semantics #366

Closed PetrJanouch closed 2 years ago

jennyowen commented 2 years ago

@PetrJanouch why did you merge this when the tests are failing?

PetrJanouch commented 2 years ago

@jennyowen the tests failed, because they did not run against the corresponding change in the mono repo. I assumed merging the two PR simultaneously will be less disruptive for the pipelines than merging the mono repo one and waiting with this PR to pick up the mono repo change to get a clean PR build.

jennyowen commented 2 years ago

@PetrJanouch oh ok. Did you run these tests on your neo4j PR then?