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

ImportTool now prints neo4j version #7057

Merged
merged 1 commit into from May 9, 2016

Conversation

burqen
Copy link
Contributor

@burqen burqen commented May 4, 2016

When importing a database, the version number of neo4j (including git branch) is printed.
This will help when user copy-paste output for support cases.

changelog: [import tool] Added print of Neo4j version

@@ -403,6 +403,7 @@ public static void main( String[] incomingArguments, boolean defaultSettingsSuit
private static void printOverview( File storeDir, Collection<Option<File[]>> nodesFiles,
Collection<Option<File[]>> relationshipsFiles )
{
System.out.println( "Neo4j version: " + Version.getKernel().getVersion() );
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a Version.getKernel().getReleaseVersion() in 2.2? If there is it might be better to use it because it gives the version without including the git hash. If there isn't, then I suggest looking for it when forward merging and using it as soon as it appears. Don't think there is any need to print out the git hash in this output.
See Mattias change at https://github.com/neo4j/neo4j/blame/2.3/community/import-tool/src/main/java/org/neo4j/tooling/ImportTool.java#L522 for something similar (but there it was included in a URI, which meant the git hash actually broke things).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

When importing a database, the release version number of neo4j is printed.
This will help when user copy-paste output for support cases.
@burqen burqen force-pushed the 2.2-import-tool-print-version branch from 3ccd9b3 to 1b638dd Compare May 4, 2016 12:52
@spacecowboy spacecowboy merged commit e91b121 into neo4j:2.2 May 9, 2016
@burqen burqen deleted the 2.2-import-tool-print-version branch May 3, 2018 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants