This is an automated email from the ASF dual-hosted git repository. jdyer pushed a commit to branch branch_9x in repository https://gitbox.apache.org/repos/asf/solr.git
The following commit(s) were added to refs/heads/branch_9x by this push: new aa6ad5f4483 Revert "SOLR-17541: Deprecate Http2SolrClient.Builder#withListenerFactory & CloudHttpSolrClient.Builder#withHttpClient (only)" aa6ad5f4483 is described below commit aa6ad5f448397e08f7059bcbbcec89fd20afac9d Author: jdyer1 <jd...@apache.org> AuthorDate: Mon Dec 23 17:43:13 2024 -0600 Revert "SOLR-17541: Deprecate Http2SolrClient.Builder#withListenerFactory & CloudHttpSolrClient.Builder#withHttpClient (only)" This reverts commit bc94ca3012de98752d88e8cd7ba3f4e4ac75ac66. --- solr/CHANGES.txt | 5 +---- .../solr/client/solrj/impl/CloudHttp2SolrClient.java | 2 -- .../solr/client/solrj/impl/Http2SolrClient.java | 20 -------------------- 3 files changed, 1 insertion(+), 26 deletions(-) diff --git a/solr/CHANGES.txt b/solr/CHANGES.txt index 458de3cae9c..e0873adb578 100644 --- a/solr/CHANGES.txt +++ b/solr/CHANGES.txt @@ -11,10 +11,7 @@ New Features Improvements --------------------- -* SOLR-17541: Deprecate `CloudHttp2SolrClient.Builder#withHttpClient` in favor of - `CloudHttp2SolrClient.Builder#withInternalClientBuilder`. - Deprecate `LBHttp2SolrClient.Builder#withListenerFactory` in favor of - `LBHttp2SolrClient.Builder#withListenerFactories` (James Dyer) +(No changes) Optimizations --------------------- diff --git a/solr/solrj/src/java/org/apache/solr/client/solrj/impl/CloudHttp2SolrClient.java b/solr/solrj/src/java/org/apache/solr/client/solrj/impl/CloudHttp2SolrClient.java index 92a47fe6380..670bd2963ff 100644 --- a/solr/solrj/src/java/org/apache/solr/client/solrj/impl/CloudHttp2SolrClient.java +++ b/solr/solrj/src/java/org/apache/solr/client/solrj/impl/CloudHttp2SolrClient.java @@ -412,9 +412,7 @@ public class CloudHttp2SolrClient extends CloudSolrClient { * * @param httpClient http client * @return this - * @deprecated Please use {@link #withInternalClientBuilder(Http2SolrClient.Builder)} */ - @Deprecated(since = "9.9") public Builder withHttpClient(Http2SolrClient httpClient) { if (this.internalClientBuilder != null) { throw new IllegalStateException( diff --git a/solr/solrj/src/java/org/apache/solr/client/solrj/impl/Http2SolrClient.java b/solr/solrj/src/java/org/apache/solr/client/solrj/impl/Http2SolrClient.java index 8e692e8cdd9..cf8415ee64c 100644 --- a/solr/solrj/src/java/org/apache/solr/client/solrj/impl/Http2SolrClient.java +++ b/solr/solrj/src/java/org/apache/solr/client/solrj/impl/Http2SolrClient.java @@ -1018,26 +1018,6 @@ public class Http2SolrClient extends HttpSolrClientBase { this.baseSolrUrl = baseSolrUrl; } - /** - * Specify listener factories, which will replace any existing values. - * - * @param listenerFactories a list of HttpListenerFactory instances - * @return This Builder - */ - public Http2SolrClient.Builder withListenerFactories( - List<HttpListenerFactory> listenerFactories) { - this.listenerFactory = listenerFactories; - return this; - } - - /** - * Specify listener factories, which will replace any existing values. - * - * @param listenerFactory a list of HttpListenerFactory instances - * @return This Builder - * @deprecated Please use {@link #withListenerFactories(List)} - */ - @Deprecated(since = "9.9") public Http2SolrClient.Builder withListenerFactory(List<HttpListenerFactory> listenerFactory) { this.listenerFactory = listenerFactory; return this;