SOLR-18122: Remove deprecated class#4525
Conversation
epugh
left a comment
There was a problem hiding this comment.
LGTM. Having said that, it feels like there should be more of a pointer to the fix, but this also seems so down in the weeds that maybe that doesn't matter. Sigh, ignore my angst. I triggered the run, and assuming it passes will merge.
There was a problem hiding this comment.
Pull request overview
Removes the deprecated ZooKeeper ACL provider VMParamsAllAndReadonlyDigestZkACLProvider from solrj-zookeeper as part of Solr 10 deprecated-code cleanup (SOLR-18122), and documents the removal for upgraders and release notes.
Changes:
- Delete deprecated
VMParamsAllAndReadonlyDigestZkACLProviderimplementation (no remaining in-repo references found). - Add Solr 10 upgrade note pointing users to
DigestZkACLProvider+VMParamsZkCredentialsInjector. - Add an unreleased changelog entry marking the class removal.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
solr/solrj-zookeeper/src/java/org/apache/solr/common/cloud/VMParamsAllAndReadonlyDigestZkACLProvider.java |
Removes deprecated ACL provider class. |
solr/solr-ref-guide/modules/upgrade-notes/pages/major-changes-in-solr-10.adoc |
Documents the removal and replacement approach for Solr 10 upgrades. |
changelog/unreleased/SOLR-18122-remove-vmparams-acl-provider.yml |
Adds changelog entry describing the removal. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Haha, I get it. I followed the 9.x convention. And did add a reference of what to use in its stead in the changelog and guide. Can also add a xref to the 9.x note but I think it's minor. Lmk if you were thinking in another direction
Thank you 🙌 |
|
@epugh, can you approve the pending workflow here? |
|
need to confirm we can remove this from |
https://issues.apache.org/jira/browse/SOLR-18122
Description
Remove the deprecated
VMParamsAllAndReadonlyDigestZkACLProviderclass as part of the Solr 10 deprecated-code cleanupChecklist
Please review the following and check all that apply:
mainbranch../gradlew check.