Java 9 build 148 causes trouble in Apache Lucene/Solr/Elasticsearch
Alan Bateman
Alan.Bateman at oracle.com
Thu Dec 15 12:24:55 UTC 2016
On 13/12/2016 21:18, Peter Levart wrote:
> I think this is OK.
>
> Just a couple of nits in test:
>
> 1. You create a static Path bob = Paths.get("bob") field, but then you
> don't use it in:
>
> 56 try (FileChannel fc = FileChannel.open(Paths.get("bob"),
> CREATE, WRITE)) {
>
Adding to Peter's comment, this can be further changed to use
Files.write(bob, srcData).
Otherwise I think the patch looks okay although it does feel like
invokeCleaner needs a warning in the javadoc, maybe being in Unsafe is
enough.
-Alan
More information about the jigsaw-dev
mailing list