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

Ability to build with Java 9 #1326

Closed
szpak opened this issue Aug 21, 2018 · 0 comments
Closed

Ability to build with Java 9 #1326

szpak opened this issue Aug 21, 2018 · 0 comments
Labels
area/java9 This belongs to the java9 compatibility theme type/bug A general bug
Milestone

Comments

@szpak
Copy link
Contributor

szpak commented Aug 21, 2018

Expected behavior

It would be good to be able to build the project with Java 9.

Actual behavior

Currently there are failing tests.

Extra notes

It is a prerequisite to the official Java 9(+) support in reactor-core (which - having Java 11 LTS knocking at the door - become an important thing). I haven't found any related issue, but I believe you have it in your plans (there is more work to do to achieve that).

Reactor Core version

current master

JVM version (e.g. java -version)

OpenJDK Runtime Environment (build 9.0.4+11)

szpak added a commit to szpak-forks/reactor-core that referenced this issue Aug 21, 2018
Due to different stacktrace.

Java 9 inserts 2 more lines in the middle:
java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
szpak added a commit to szpak-forks/reactor-core that referenced this issue Aug 21, 2018
Casting even empty Objects[] to Integer[] results in:
java.lang.ClassCastException: java.base/[Ljava.lang.Object; cannot be cast to java.base/[Ljava.lang.Integer;
(verified by QueuesTest.toArrayOnZeroQueueShouldNotFailAlsoOnJava9() and
zeroQueueOperations()).

For empty queue. The production implementation was changed (and also slightly
simplified).

The same problem is also with the other Queue implementations. Will be
fixed separately.
szpak added a commit to szpak-forks/reactor-core that referenced this issue Aug 21, 2018
szpak added a commit to szpak-forks/reactor-core that referenced this issue Aug 21, 2018
@simonbasle simonbasle added this to the 3.2.0.RC1 milestone Aug 22, 2018
@simonbasle simonbasle added type/bug A general bug area/java9 This belongs to the java9 compatibility theme labels Aug 22, 2018
szpak added a commit to szpak-forks/reactor-core that referenced this issue Aug 22, 2018
It was failing with ClassCastException on toArray([]) conversion.

More tests added.
szpak added a commit to szpak-forks/reactor-core that referenced this issue Aug 22, 2018
It was failing with ClassCastException on toArray([]) conversion.

More tests added.
szpak added a commit to szpak-forks/reactor-core that referenced this issue Aug 22, 2018
It was failing with ClassCastException on toArray([]) conversion.

More tests added.
szpak added a commit to szpak-forks/reactor-core that referenced this issue Aug 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/java9 This belongs to the java9 compatibility theme type/bug A general bug
Projects
None yet
Development

No branches or pull requests

2 participants