> On 2 Mar 2016, at 21:37, Martin Buchholz <marti...@google.com> wrote: > > Webrevs updated, incorporating changes to tests in my previous message.
Looks ok, but i went through rather quickly. java/util/concurrent/ScheduledThreadPoolExecutor/DelayOverflow.java — - pool.schedule(keepPoolBusy, 0, TimeUnit.SECONDS); + pool.schedule(keepPoolBusy, 0, DAYS); It probably does not matter that you changed the units here? Paul.
