Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update neo4j-java-driver from 1.7.5 to 4.0.1 #594

Merged
merged 1 commit into from
May 26, 2020

Conversation

michaelweidmann
Copy link
Contributor

@michaelweidmann michaelweidmann commented May 26, 2020

Description

This pull request updates the Neo4j Client library to the newest version to support tests with Neo4j 4.X databases and the new neo4j:// URI scheme which provides a much better way to connect to Neo4j databases.

Motivation and Context

This change is required to test Neo4j databases with a version higher than 3 and to test them more reliably with new URI schemes.

How Has This Been Tested?

./gradlew check was successful:

BUILD SUCCESSFUL in 4m 13s
259 actionable tasks: 50 executed, 209 up-to-date

Types of changes

New feature:

  • Bumping up the Neo4j Driver version
  • Needed to add a license to src/licenses/licenses/reactive-streams/LICENSE because the new version of the library uses reactive streams

Breaking Changes:

  • The interface of the library changed so I updated the imports and the classes in the code
  • A test used a method which does not exist anymore so I updated the test

Checklist:

  • My code follows the [code style][style-guide] of this project.
  • I have updated the documentation accordingly.

@michaelweidmann
Copy link
Contributor Author

I honestly don't know why the Travis CI pipeline crashes...

@pmouawad pmouawad merged commit cc499d7 into apache:master May 26, 2020
asfgit pushed a commit that referenced this pull request May 28, 2020
Bump version to 5.3.1 and copy changes to old_changes
Keep the changelog entry from pr #594 as it was added after the last release.
kkalinin pushed a commit to kkalinin/jmeter that referenced this pull request Mar 11, 2021
kkalinin pushed a commit to kkalinin/jmeter that referenced this pull request Mar 11, 2021
Bump version to 5.3.1 and copy changes to old_changes
Keep the changelog entry from pr apache#594 as it was added after the last release.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants