Migrate SolrJ client APIs for 10.0.0 - #3988
Closed
skrcode wants to merge 1 commit into
Closed
Conversation
Signed-off-by: Suraj Rajan <surajkrajan95@gmail.com>
Collaborator
|
Hi @skrcode, thank your for your contribution. We generally avoid unsolicited code from unknown third parties. Please refrain from creating pull requests in the future, unless you've been in contact with us. Best, |
Author
|
sure, no worries. thanks anyways ! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Completes #3953 by migrating the backend to the SolrJ 10 client API:
solr-solrj-jettyartifact and use the renamed Jetty clientimplementations
SolrQueryand streaming response types to their SolrJ 10 packagesDocumentObjectBinder.INSTANCEVerification (JDK 21):
mvn -q -pl backend -am clean compile -DskipTests— passedmvn -q -pl backend test-compile -DskipTests— passedmvn -q -pl backend test— 1,121 tests executed with 0 assertionfailures; the run could not complete because the disposable environment has
no Docker daemon, producing two Testcontainers setup errors for HANA and
ClickHouse
git diff --check— passedDevelopment assistance: prepared in Codex with the maintainer-intent,
fast-execution, remote-Java, and diff-review skills from
JAIPilot. The exact source was executed
in a disposable remote workspace that was destroyed after the run. Execution profile:
gpt-5.6-sol · xhigh · fast.