[BOOKKEEPER-1049] Run BookKeeper on Java9

• Type: Improvement
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Fix Version/s: 4.6.0
• Labels: ghmigrated iktriage0

Environment

java9

• Assignee: Enrico Olivelli
• Reporter: Enrico Olivelli
• Created: 08/May/17 13:10
• Updated: 17/Oct/17 21:26

Description

I am tring to run BookKeeper on Java9, a part from simple issues:

• exclude the transitive dependency to jdk.tools in bookkeeer-benchmarks
• fix a warning due to the fact the Class.forName in java9 is deprecated and we are running javac with the "Werror" option (warning = error)

we have a real stopper, in fact we want to access the low level ID of FileDescriptors using the reflaction and by default this is not allowed.

This is the error:

java.lang.AssertionError
        at org.apache.bookkeeper.util.NativeIO.getFieldByReflection(NativeIO.java:63)
        at org.apache.bookkeeper.util.NativeIO.getSysFileDescriptor(NativeIO.java:75)
        at org.apache.bookkeeper.bookie.JournalChannel.<init>(JournalChannel.java:218)
        at org.apache.bookkeeper.bookie.JournalChannel.<init>(JournalChannel.java:97)
        at org.apache.bookkeeper.bookie.JournalChannel.<init>(JournalChannel.java:86)
        at org.apache.bookkeeper.bookie.UpgradeTest.writeJournal(UpgradeTest.java:83)
        at org.apache.bookkeeper.bookie.UpgradeTest.newV1JournalDirectory(UpgradeTest.java:114)
        at org.apache.bookkeeper.bookie.UpgradeTest.testUpgradeV1toCurrent(UpgradeTest.java:192)

Issue Links

• depends upon: BOOKKEEPER-816 use native fallocate & sync_file_range to improve journal allocation