-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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
1.x: javac 9 compatibility fixes #3824
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
👍 // @akarnokd maybe report "incorrect" behavior of JDK9 compiler to OpenJDK issue tracker? |
There were reports such as this before. Bottom line is that they are fine with their breaking change. |
👍 |
busbey
added a commit
to busbey/YCSB
that referenced
this pull request
Apr 16, 2016
* in couchbase2 work around ReactiveX/RxJava#3824 * accumulo test hangs, so skip tests * hbase10 test crashes with unknown class version, so skip tests * core jar-with-deps fails, so keep them broken out fixes brianfrankcooper#705
busbey
added a commit
to busbey/YCSB
that referenced
this pull request
Apr 16, 2016
* in couchbase2 work around ReactiveX/RxJava#3824 * accumulo test hangs, so skip tests * hbase10 test crashes with unknown class version, so skip tests * core jar-with-deps fails, so keep them broken out * add jdk9 to travis fixes brianfrankcooper#705
busbey
added a commit
to busbey/YCSB
that referenced
this pull request
Apr 16, 2016
* in couchbase2 work around ReactiveX/RxJava#3824 * accumulo test hangs, so skip tests * hbase10 test crashes with unknown class version, so skip tests * core jar-with-deps fails, so keep them broken out * add jdk9 to travis fixes brianfrankcooper#704, fixes brianfrankcooper#705
busbey
added a commit
to busbey/YCSB
that referenced
this pull request
Apr 16, 2016
* in couchbase2 work around ReactiveX/RxJava#3824 * accumulo test hangs, so skip tests * hbase10 test crashes with unknown class version, so skip tests * core jar-with-deps fails, so keep them broken out * add jdk9 to travis fixes brianfrankcooper#704, fixes brianfrankcooper#705
busbey
added a commit
to busbey/YCSB
that referenced
this pull request
Apr 17, 2016
* in couchbase2 work around ReactiveX/RxJava#3824 * accumulo test hangs, so skip tests * hbase10 test crashes with unknown class version, so skip tests * core jar-with-deps fails, so keep them broken out fixes brianfrankcooper#704, fixes brianfrankcooper#705
busbey
added a commit
to busbey/YCSB
that referenced
this pull request
Apr 30, 2016
…xes for jdk9 * in couchbase2 work around ReactiveX/RxJava#3824 * accumulo test hangs, so skip tests * hbase10 and asynchbase test crashes with unknown class version, so skip tests * orientdb tests fail, so skip * core jar-with-deps fails, so keep them broken out fixes brianfrankcooper#704, fixes brianfrankcooper#705
busbey
added a commit
to brianfrankcooper/YCSB
that referenced
this pull request
Sep 30, 2016
…xes for jdk9 (#712) * in couchbase2 work around ReactiveX/RxJava#3824 * accumulo test hangs, so skip tests * hbase10 and asynchbase test crashes with unknown class version, so skip tests * orientdb tests fail, so skip * core jar-with-deps fails, so keep them broken out fixes #704, fixes #705
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The type inference of javac in JDK 9 has been changed in an incompatible way, marking many generics-related code invalid.
The most common problems:
Observable<? extends T>
but from its perspective, a call tosubscribe()
with aSubscriber<T>
is ambiguous as it matchessubscribe(Observer<? super T>)
andsubscribe(Subscriber<? super T>)
. Somehow, the more specific class is not considered as a valid choice. Downcasting toObservable<T>
fixes the error.A<?>[]
no longer acceptsA<Ti>
elements, requires the use of raw types to get aroundThe tests appear to compile fine.
Note that I couldn't find a working IDE for JDK 9 and had to revert to command line trickery: