Skip to content

SOLR-18166: Remove 'linkconfig' tool and related code#4232

Open
gerlowskija wants to merge 2 commits intoapache:mainfrom
gerlowskija:SOLR-18166-wip
Open

SOLR-18166: Remove 'linkconfig' tool and related code#4232
gerlowskija wants to merge 2 commits intoapache:mainfrom
gerlowskija:SOLR-18166-wip

Conversation

@gerlowskija
Copy link
Contributor

https://issues.apache.org/jira/browse/SOLR-18166

Description

"Linkconfig" is a functionality initially added prior to our collection-admin APIs, when users needed to run manual commands to associate or "link" a new collection with a particular config by storing the config name on the /collections/ node.

Storing config information in this location was deprecated in SOLR-14341 and can be removed now that 10.0 is out the door.

Solution

This commit removes several methods that checked for config info at this location, as well as the "linkconfig" that was user's primary way of storing this information in the past.

Tests

Modifies ZkSubcommandsTest, but doesn't add any net-new tests as this PR is mainly removing dead code rather than adding new functionality.

Checklist

Please review the following and check all that apply:

  • I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
  • I have created a Jira issue and added the issue ID to my pull request title.
  • I have given Solr maintainers access to contribute to my PR branch. (optional but recommended, not available for branches on forks living under an organisation)
  • I have developed this patch against the main branch.
  • I have run ./gradlew check.
  • I have added tests for my changes.
  • I have added documentation for the Reference Guide
  • I have added a changelog entry for my change

"Linkconfig" is a functionality initially added prior to our
collection-admin APIs, when users needed to run manual commands to
associate or "link" a new collection with a particular config by storing
the config name on the /collections/<collName> node.

Storing config information in this location was deprecated in SOLR-14341
and can be removed now that 10.0 is out the door.  This commit removes
several methods that checked for config info at this location, as well
as the "linkconfig" that was user's primary way of storing this
information in the past.
@gerlowskija gerlowskija marked this pull request as ready for review March 20, 2026 23:21
Copy link
Contributor

@epugh epugh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I was surprised nothing in bin/solr and ref guide needed removing?

* @return the ClusterState
*/
@SuppressWarnings({"unchecked"})
@Deprecated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yay! finally

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes!

Copy link
Contributor

@dsmiley dsmiley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great removal!

* @return the ClusterState
*/
@SuppressWarnings({"unchecked"})
@Deprecated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants