Skip to content

Migrate SolrJ client APIs for 10.0.0 - #3988

Closed
skrcode wants to merge 1 commit into
ingef:dependabot/maven/org.apache.solr-solr-solrj-10.0.0from
skrcode:jaipilot/fix-dependabot-3953
Closed

Migrate SolrJ client APIs for 10.0.0#3988
skrcode wants to merge 1 commit into
ingef:dependabot/maven/org.apache.solr-solr-solrj-10.0.0from
skrcode:jaipilot/fix-dependabot-3953

Conversation

@skrcode

@skrcode skrcode commented Aug 30, 2026

Copy link
Copy Markdown

Completes #3953 by migrating the backend to the SolrJ 10 client API:

  • add the split solr-solrj-jetty artifact and use the renamed Jetty client
    implementations
  • move SolrQuery and streaming response types to their SolrJ 10 packages
  • replace the removed client binder accessor with DocumentObjectBinder.INSTANCE

Verification (JDK 21):

  • mvn -q -pl backend -am clean compile -DskipTests — passed
  • mvn -q -pl backend test-compile -DskipTests — passed
  • remote mvn -q -pl backend test — 1,121 tests executed with 0 assertion
    failures; 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 — passed

Development 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.

Signed-off-by: Suraj Rajan <surajkrajan95@gmail.com>
@awildturtok

Copy link
Copy Markdown
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,
awildturtok

@skrcode

skrcode commented Sep 1, 2026

Copy link
Copy Markdown
Author

sure, no worries. thanks anyways !

@skrcode skrcode closed this Sep 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants