Integrated: 8324673: javacserver failed during build: RejectedExecutionException

[Daniel Jeliński]

On Mon, 8 Apr 2024 09:20:44 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:

> The RejectedExecutionException was thrown when the thread executing `Server.start` managed to shut down the `compilerThreadPool` before the thread executing `Server.handleRequest` submitted the compilation task.
> 
> This patch removes the extra thread used for `Server.handleRequest`, and executes that method directly in the thread pool. All `compilerThreadPool` uses happen on the `Server.start` thread now, and no new tasks are submitted after the thread pool is shut down.
> 
> In order to verify the fix, I modified `IdleMonitor.KEEPALIVE` to 1 second. With that change the problem was occasionally reproducible without the patch from this PR. With the patch, the `RejectedExecutionException` problem did not reproduce. 
> 
> No new regression test. Existing langtools tests continue to pass.

This pull request has now been integrated.

Changeset: 3b6629ce
Author:    Daniel Jeliński <djelinski at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/3b6629cec7a2ecec8dcb5b94d8ed3e169483aa97
Stats:     19 lines in 2 files changed: 1 ins; 10 del; 8 mod

8324673: javacserver failed during build: RejectedExecutionException

Reviewed-by: cstein, erikj

-------------

PR: https://git.openjdk.org/jdk/pull/18672