Issue #12023 - Remove deprecated classes/methods.

Removed deprecated methods from AbstractConnectionPool.

Signed-off-by: Simone Bordet <[email protected]>

Author: sbordet

jetty-core/jetty-client/src/main/java/org/eclipse/jetty/client/AbstractConnectionPool.java

@@ -307,7 +307,6 @@ public boolean accept(Connection connection)
         attachable.setAttachment(new EntryHolder(entry));
         onCreated(connection);
         entry.enable(connection, false);
-        idle(connection, false);
         return true;
     }
 
@@ -370,7 +369,6 @@ protected Connection activate()
 
                 if (LOG.isDebugEnabled())
                     LOG.debug("Activated {} {}", entry, pool);
-                acquired(connection);
                 return connection;
             }
             return null;
@@ -391,10 +389,7 @@ public boolean isActive(Connection connection)
     @Override
     public boolean release(Connection connection)
     {
-        if (!deactivate(connection))
-            return false;
-        released(connection);
-        return idle(connection, isStopped());
+        return deactivate(connection);
     }
 
     protected boolean deactivate(Connection connection)
@@ -443,10 +438,7 @@ public boolean remove(Connection connection)
         if (LOG.isDebugEnabled())
             LOG.debug("Removed ({}) {} {}", removed, holder.entry, pool);
         if (removed)
-        {
-            released(connection);
             onRemoved(connection);
-        }
         return removed;
     }
 
@@ -460,45 +452,6 @@ protected void onCreated(Connection connection)
     {
     }
 
-    /**
-     * @param connection the {@link Connection} that become idle
-     * @param close whether this pool is closing
-     * @return {@code true} to indicate that the connection is idle, {@code false} otherwise
-     * @deprecated Racy API. Do not use. There is no replacement.
-     */
-    @Deprecated(since = "12.0.8", forRemoval = true)
-    protected boolean idle(Connection connection, boolean close)
-    {
-        return !close;
-    }
-
-    /**
-     * @param connection the {@link Connection} that was acquired
-     * @deprecated Racy API. Do not use. There is no replacement.
-     */
-    @Deprecated(since = "12.0.8", forRemoval = true)
-    protected void acquired(Connection connection)
-    {
-    }
-
-    /**
-     * @param connection the {@link Connection} that was released
-     * @deprecated Racy API. Do not use. There is no replacement.
-     */
-    @Deprecated(since = "12.0.8", forRemoval = true)
-    protected void released(Connection connection)
-    {
-    }
-
-    /**
-     * @param connection the {@link Connection} that was removed
-     * @deprecated replaced by {@link #onRemoved(Connection)}
-     */
-    @Deprecated(since = "12.0.8", forRemoval = true)
-    protected void removed(Connection connection)
-    {
-    }
-
     /**
      * <p>Callback method invoked when a {@link Connection} has been removed from this pool.</p>
      *
@@ -507,7 +460,6 @@ protected void removed(Connection connection)
      */
     protected void onRemoved(Connection connection)
     {
-        removed(connection);
     }
 
     Collection<Connection> getIdleConnections()
@@ -621,7 +573,6 @@ public void succeeded(Connection connection)
                 onCreated(connection);
                 pending.decrementAndGet();
                 reserved.enable(connection, false);
-                idle(connection, false);
                 super.succeeded(connection);
                 proceed();
             }

jetty-core/jetty-client/src/main/java/org/eclipse/jetty/client/ValidatingConnectionPool.java

@@ -80,8 +80,6 @@ public boolean release(Connection connection)
         quarantine.put(connection, holder);
         if (LOG.isDebugEnabled())
             LOG.debug("Validating for {}ms {}", timeout, connection);
-
-        released(connection);
         return true;
     }
 
@@ -135,7 +133,6 @@ public void run()
                 quarantine.remove(connection);
                 if (!closed)
                     deactivate(connection);
-                idle(connection, closed);
                 proceed();
             }
         }