From 738717ef5201354092c6e8c4ea1e691d3913cab7 Mon Sep 17 00:00:00 2001 From: Gian Merlino Date: Mon, 14 Apr 2025 23:01:48 -0700 Subject: [PATCH 1/7] k8s: Update fabric8 to 6.13.1, switch to Vert.x http client. fabric8 6.x uses the OkHttp client by default, which itself uses an unbounded-size cached pool, with 60s timeout, for making requests. In production with large task counts, this leads to lots of thread churn. The Vert.x client is the default in fabric8 7.x, and is better behaved. By default it uses a worker pool of 20 threads (see javadoc for VertxOptions). This patch uses fabric8 6.13.1 rather than some later version, because higher versions are not compatible with Jackson 2.12. --- .../kubernetes-overlord-extensions/pom.xml | 28 +++++++++++++++---- .../common/DruidKubernetesClient.java | 21 ++++++-------- .../KubernetesTaskRunnerFactoryTest.java | 3 +- .../DruidPeonClientIntegrationTest.java | 3 +- 4 files changed, 36 insertions(+), 19 deletions(-) diff --git a/extensions-core/kubernetes-overlord-extensions/pom.xml b/extensions-core/kubernetes-overlord-extensions/pom.xml index eb2fc63f8b2f..ba2e213f6d3b 100644 --- a/extensions-core/kubernetes-overlord-extensions/pom.xml +++ b/extensions-core/kubernetes-overlord-extensions/pom.xml @@ -34,6 +34,13 @@ ../../pom.xml + + + 6.13.1 + +