Merge pull request quarkusio#17425 from Sanne/RemovingThreadLocalPgPool

Remove the ThreadLocalPgPool implementation: vertx 4 has a Pool which doesn't need wrapping
gsmet · May 24, 2021 · 2c4c3b2 · 2c4c3b2
2 parents d9dc83f + 0cc0a84
commit 2c4c3b2
Show file tree

Hide file tree

Showing 10 changed files with 4 additions and 643 deletions.
diff --git a/...yment/src/test/java/io/quarkus/reactive/datasource/runtime/ConnectionPoolsClosedTest.java b/...yment/src/test/java/io/quarkus/reactive/datasource/runtime/ConnectionPoolsClosedTest.java
diff --git a/...loyment/src/test/java/io/quarkus/reactive/datasource/runtime/TestableThreadLocalPool.java b/...loyment/src/test/java/io/quarkus/reactive/datasource/runtime/TestableThreadLocalPool.java
diff --git a/...src/main/java/io/quarkus/reactive/datasource/runtime/DataSourceReactiveRuntimeConfig.java b/...src/main/java/io/quarkus/reactive/datasource/runtime/DataSourceReactiveRuntimeConfig.java
@@ -28,7 +28,6 @@ public class DataSourceReactiveRuntimeConfig {
 
     /**
      * The datasource pool maximum size.
-     * Note that a separate pool instance is started for each thread using it: the size limits each individual pool instance.
      */
     @ConfigItem
     public OptionalInt maxSize = OptionalInt.empty();
@@ -88,8 +87,7 @@ public class DataSourceReactiveRuntimeConfig {
     public PfxConfiguration keyCertificatePfx = new PfxConfiguration();
 
     /**
-     * Deprecated: this will be removed with no replacement.
-     * We always return a threadsafe pool now, using a separate Pool instance for each Thread.
+     * Deprecated: this was removed and is no longer available.
      * 
      * @Deprecated
      */