Skip to content

Commit cf7f507

Browse files
authored
SOLR-17962 Move deprecated SolrClient impls (Apache HttpClient) to test-framework (#3793)
They will eventually be deleted after tests are migrated. Move to package org.apache.solr.client.solrj.apache. Uncovered a couple trivial visibility matters. * ignore ConnectionReuseTest -- we're going to delete soon anyway
1 parent e46e72c commit cf7f507

File tree

162 files changed

+305
-257
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

162 files changed

+305
-257
lines changed
Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
# See https://github.com/apache/solr/blob/main/dev-docs/changelog.adoc
2+
title: The deprecated SolrClient implementations based on Apache HttpClient are removed from Solrj,
3+
thus the related dependencies are no longer present.
4+
They temporarily live on in the test framework.
5+
type: other # added, changed, fixed, deprecated, removed, dependency_update, security, other
6+
authors:
7+
- name: David Smiley
8+
links:
9+
- name: SOLR-17962
10+
url: https://issues.apache.org/jira/browse/SOLR-17962
11+
issues:
12+
- 17962

solr/core/gradle.lockfile

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ com.jayway.jsonpath:json-path:2.9.0=compileClasspath,jarValidation,runtimeClassp
3636
com.microsoft.onnxruntime:onnxruntime:1.23.0=jarValidation,testRuntimeClasspath
3737
com.tdunning:t-digest:3.3=compileClasspath,jarValidation,runtimeClasspath,runtimeLibs,testCompileClasspath,testRuntimeClasspath
3838
commons-cli:commons-cli:1.10.0=compileClasspath,jarValidation,runtimeClasspath,runtimeLibs,testCompileClasspath,testRuntimeClasspath
39-
commons-codec:commons-codec:1.19.0=apiHelper,compileClasspath,jarValidation,runtimeClasspath,runtimeLibs,testCompileClasspath,testRuntimeClasspath
39+
commons-codec:commons-codec:1.19.0=compileClasspath,jarValidation,runtimeClasspath,runtimeLibs,testCompileClasspath,testRuntimeClasspath
4040
commons-io:commons-io:2.20.0=apiHelper,compileClasspath,jarValidation,runtimeClasspath,runtimeLibs,testCompileClasspath,testRuntimeClasspath
4141
io.dropwizard.metrics:metrics-annotation:4.2.26=jarValidation,testRuntimeClasspath
4242
io.dropwizard.metrics:metrics-core:4.2.26=compileClasspath,jarValidation,runtimeClasspath,runtimeLibs,testCompileClasspath,testRuntimeClasspath
@@ -92,9 +92,9 @@ org.apache.commons:commons-math3:3.6.1=apiHelper,compileClasspath,jarValidation,
9292
org.apache.curator:curator-client:5.9.0=apiHelper,compileClasspath,jarValidation,runtimeClasspath,runtimeLibs,testCompileClasspath,testRuntimeClasspath
9393
org.apache.curator:curator-framework:5.9.0=apiHelper,compileClasspath,jarValidation,runtimeClasspath,runtimeLibs,testCompileClasspath,testRuntimeClasspath
9494
org.apache.curator:curator-test:5.9.0=jarValidation,testCompileClasspath,testRuntimeClasspath
95-
org.apache.httpcomponents:httpclient:4.5.14=apiHelper,jarValidation,runtimeClasspath,runtimeLibs,testCompileClasspath,testRuntimeClasspath
96-
org.apache.httpcomponents:httpcore:4.4.16=apiHelper,jarValidation,runtimeClasspath,runtimeLibs,testCompileClasspath,testRuntimeClasspath
97-
org.apache.httpcomponents:httpmime:4.5.14=apiHelper,jarValidation,runtimeClasspath,runtimeLibs,testRuntimeClasspath
95+
org.apache.httpcomponents:httpclient:4.5.14=jarValidation,testCompileClasspath,testRuntimeClasspath
96+
org.apache.httpcomponents:httpcore:4.4.16=jarValidation,testCompileClasspath,testRuntimeClasspath
97+
org.apache.httpcomponents:httpmime:4.5.14=jarValidation,testRuntimeClasspath
9898
org.apache.logging.log4j:log4j-api:2.21.0=compileClasspath,jarValidation,runtimeClasspath,runtimeLibs,testCompileClasspath,testRuntimeClasspath
9999
org.apache.logging.log4j:log4j-core:2.21.0=compileClasspath,jarValidation,runtimeClasspath,runtimeLibs,testCompileClasspath,testRuntimeClasspath
100100
org.apache.logging.log4j:log4j-slf4j2-impl:2.21.0=jarValidation,runtimeClasspath,runtimeLibs,testRuntimeClasspath

solr/core/src/test/org/apache/solr/cloud/AliasIntegrationTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,8 @@
3838
import org.apache.solr.client.solrj.SolrQuery;
3939
import org.apache.solr.client.solrj.SolrRequest;
4040
import org.apache.solr.client.solrj.SolrServerException;
41+
import org.apache.solr.client.solrj.apache.CloudLegacySolrClient;
4142
import org.apache.solr.client.solrj.cloud.SolrCloudManager;
42-
import org.apache.solr.client.solrj.impl.CloudLegacySolrClient;
4343
import org.apache.solr.client.solrj.impl.CloudSolrClient;
4444
import org.apache.solr.client.solrj.impl.ClusterStateProvider;
4545
import org.apache.solr.client.solrj.impl.ZkClientClusterStateProvider;

solr/core/src/test/org/apache/solr/cloud/BalanceReplicasTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@
3434
import org.apache.http.entity.ContentType;
3535
import org.apache.http.util.EntityUtils;
3636
import org.apache.solr.client.api.model.BalanceReplicasRequestBody;
37-
import org.apache.solr.client.solrj.impl.CloudLegacySolrClient;
37+
import org.apache.solr.client.solrj.apache.CloudLegacySolrClient;
3838
import org.apache.solr.client.solrj.impl.CloudSolrClient;
3939
import org.apache.solr.client.solrj.request.CollectionAdminRequest;
4040
import org.apache.solr.common.cloud.DocCollection;

solr/core/src/test/org/apache/solr/cloud/ChaosMonkeyNothingIsSafeWithPullReplicasTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
import org.apache.lucene.tests.util.LuceneTestCase;
2727
import org.apache.solr.SolrTestCaseJ4.SuppressSSL;
2828
import org.apache.solr.client.solrj.SolrQuery;
29-
import org.apache.solr.client.solrj.impl.CloudLegacySolrClient;
29+
import org.apache.solr.client.solrj.apache.CloudLegacySolrClient;
3030
import org.apache.solr.client.solrj.impl.CloudSolrClient;
3131
import org.apache.solr.common.SolrInputDocument;
3232
import org.apache.solr.common.cloud.DocCollection;

solr/core/src/test/org/apache/solr/cloud/CreateRoutedAliasTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@
3737
import org.apache.http.util.EntityUtils;
3838
import org.apache.solr.SolrTestCaseJ4;
3939
import org.apache.solr.client.solrj.SolrServerException;
40-
import org.apache.solr.client.solrj.impl.CloudLegacySolrClient;
40+
import org.apache.solr.client.solrj.apache.CloudLegacySolrClient;
4141
import org.apache.solr.client.solrj.impl.CloudSolrClient;
4242
import org.apache.solr.client.solrj.request.CollectionAdminRequest;
4343
import org.apache.solr.client.solrj.response.CollectionAdminResponse;

solr/core/src/test/org/apache/solr/cloud/ForceLeaderTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,8 @@
2424
import java.util.Optional;
2525
import java.util.concurrent.TimeUnit;
2626
import org.apache.solr.client.solrj.SolrServerException;
27+
import org.apache.solr.client.solrj.apache.CloudLegacySolrClient;
2728
import org.apache.solr.client.solrj.cloud.SocketProxy;
28-
import org.apache.solr.client.solrj.impl.CloudLegacySolrClient;
2929
import org.apache.solr.client.solrj.impl.CloudSolrClient;
3030
import org.apache.solr.client.solrj.request.CollectionAdminRequest;
3131
import org.apache.solr.common.SolrException;

solr/core/src/test/org/apache/solr/cloud/FullSolrCloudDistribCmdsTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,9 @@
3030
import org.apache.lucene.tests.util.TestUtil;
3131
import org.apache.solr.client.solrj.SolrClient;
3232
import org.apache.solr.client.solrj.SolrQuery;
33+
import org.apache.solr.client.solrj.apache.ConcurrentUpdateSolrClient;
3334
import org.apache.solr.client.solrj.cloud.SocketProxy;
3435
import org.apache.solr.client.solrj.impl.CloudSolrClient;
35-
import org.apache.solr.client.solrj.impl.ConcurrentUpdateSolrClient;
3636
import org.apache.solr.client.solrj.request.CollectionAdminRequest;
3737
import org.apache.solr.client.solrj.request.UpdateRequest;
3838
import org.apache.solr.client.solrj.response.RequestStatusState;

solr/core/src/test/org/apache/solr/cloud/HttpPartitionTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,8 +36,8 @@
3636
import org.apache.solr.SolrTestCaseJ4.SuppressSSL;
3737
import org.apache.solr.client.solrj.SolrClient;
3838
import org.apache.solr.client.solrj.SolrServerException;
39+
import org.apache.solr.client.solrj.apache.HttpSolrClient;
3940
import org.apache.solr.client.solrj.cloud.SocketProxy;
40-
import org.apache.solr.client.solrj.impl.HttpSolrClient;
4141
import org.apache.solr.client.solrj.request.QueryRequest;
4242
import org.apache.solr.client.solrj.request.UpdateRequest;
4343
import org.apache.solr.common.SolrException;

solr/core/src/test/org/apache/solr/cloud/LeaderElectionContextKeyTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
import java.util.concurrent.TimeUnit;
2424
import org.apache.solr.client.solrj.SolrClient;
2525
import org.apache.solr.client.solrj.SolrServerException;
26-
import org.apache.solr.client.solrj.impl.HttpSolrClient;
26+
import org.apache.solr.client.solrj.apache.HttpSolrClient;
2727
import org.apache.solr.client.solrj.request.CollectionAdminRequest;
2828
import org.apache.solr.client.solrj.request.CoreAdminRequest;
2929
import org.apache.solr.common.cloud.ClusterState;

0 commit comments

Comments
 (0)