Skip to content

Support in-dev Neo4j version #314

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

Merged
merged 1 commit into from
Dec 18, 2017
Merged

Support in-dev Neo4j version #314

merged 1 commit into from
Dec 18, 2017

Conversation

lutovich
Copy link
Contributor

Driver receives Neo4j version in response for INIT message. Released versions have format Neo4j/X.Y.Z and in-dev version has Neo4j/dev. Previously driver was not able to parse the later and failed.

This PR fixes the problem, ServerVersion will now be able to represent an in-dev version. Such ServerVersion is treated lower than any other version because driver can't know which features it supports and which it does not.

Driver receives Neo4j version in response for INIT message. Released
versions have format `Neo4j/X.Y.Z` and in-dev version has `Neo4j/dev`.
Previously driver was not able to parse the later and failed.

This commit fixes the problem, `ServerVersion` will now be able to
represent an in-dev version. Such `ServerVersion` is treated lower
than any other version because driver can't know which features it
supports and which it does not.
@lutovich lutovich requested a review from zhenlineo December 18, 2017 13:06
@lutovich lutovich changed the base branch from 1.6 to 1.5 December 18, 2017 13:08
@zhenlineo zhenlineo merged commit b1c9d50 into neo4j:1.5 Dec 18, 2017
@lutovich lutovich deleted the 1.5-dev-db branch December 18, 2017 14:26
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