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

neo4j-analytics fails with JDK21 #408

Closed
lbulej opened this issue Sep 21, 2023 · 1 comment
Closed

neo4j-analytics fails with JDK21 #408

lbulej opened this issue Sep 21, 2023 · 1 comment
Labels
compatibility Relates to platform or system compatibility

Comments

@lbulej
Copy link
Member

lbulej commented Sep 21, 2023

The benchmark was previously limited to JDK15. After update to Neo4j 4.4.25, I was able to lift the restriction (PR coming), but not completely—the benchmark fails with JDK21 with the following chain of exceptions (see neo4j.log):

java.lang.RuntimeException: Error starting Neo4j database server at [...]
Caused by: org.neo4j.kernel.lifecycle.LifecycleException: Component 'org.neo4j.dbms.database.DefaultSystemGraphInitializer@6afbe6a1' was successfully initialized, but failed to start. Please see the attached cause exception "null".
Caused by: org.neo4j.graphdb.DatabaseShutdownException: This database is shutdown.
Caused by: java.lang.RuntimeException: Failed to wrap pointer in ByteBuffer.
Caused by: java.lang.UnsupportedOperationException
	at java.base/java.lang.invoke.VarForm.getMemberName(VarForm.java:118)
	at org.neo4j.internal.unsafe.UnsafeUtil.initDirectByteBuffer(UnsafeUtil.java:665)
	at org.neo4j.internal.unsafe.UnsafeUtil.newDirectByteBuffer(UnsafeUtil.java:648)
	at org.neo4j.io.pagecache.impl.SingleFilePageSwapper.convertToByteBuffers(SingleFilePageSwapper.java:518)

The same code works just fine with JDK20. For now, I'll limit supported JVM in neoj4-analytics to version 20 (better than 15), but version 21 is a no-go. I did not try with Neo4j 5.x, because the benchmark code needs to be updated to use it.

@lbulej lbulej added the compatibility Relates to platform or system compatibility label Sep 21, 2023
@lbulej
Copy link
Member Author

lbulej commented Oct 12, 2023

Fixed by moving to Neo4j 5.12 (#422)

@lbulej lbulej closed this as completed Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compatibility Relates to platform or system compatibility
Projects
None yet
Development

No branches or pull requests

1 participant