Hi, Looks good.
It’s trivial but we probably require a CCC for ScheduledThreadPoolExecutor calling out the NPE for null unit argument values, to clarify pool sizes of TPE, and to clarify that AbortPolicy default handler for TPE *and* STPE. We are not quite there yet with the CCC being open (soon i hope!), so i can create the CCC for you. Paul. > On 30 Mar 2017, at 14:38, Martin Buchholz <marti...@google.com> wrote: > > OK, this wave is open for review again. > > On Wed, Mar 29, 2017 at 12:57 PM, Martin Buchholz <marti...@google.com> wrote: > Oops. Martin forgot to check ScheduledExecutorService, and there's some > overlap in specs, which requires some rework. > > On Wed, Mar 29, 2017 at 11:43 AM, Martin Buchholz <marti...@google.com> wrote: > First jdk10 integration > > http://cr.openjdk.java.net/~martin/webrevs/openjdk10/jsr166-integration/ > > Mostly focused on thread pools. > >
