Skip to content

Remove deprecated code in GraphHandler for registering expressions#4100

Merged
epugh merged 3 commits intoapache:mainfrom
epugh:remove_deprecated_graphhandler_registration_support
Feb 5, 2026
Merged

Remove deprecated code in GraphHandler for registering expressions#4100
epugh merged 3 commits intoapache:mainfrom
epugh:remove_deprecated_graphhandler_registration_support

Conversation

@epugh
Copy link
Contributor

@epugh epugh commented Feb 4, 2026

Description

Remove deprecated code

Solution

Delete code. I also looked for if streamFunctions existed elsewhere but didn't find any.

Tests

Run tests

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request removes deprecated code from GraphHandler that supported the old streamFunctions configuration method for registering streaming expressions, which was deprecated in Solr 8.5.

Changes:

  • Removed deprecated streamFunctions configuration handling code from GraphHandler.inform() method
  • Removed unused imports that were only needed for the deprecated code
  • Added changelog entry documenting the removal

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
solr/core/src/java/org/apache/solr/handler/GraphHandler.java Removed deprecated streamFunctions configuration code block and associated imports (Collections, Map.Entry, Expressible, NamedList, PluginInfo, SolrConfig)
changelog/unreleased/PR#4100-remove-deprecated-graphhandler-registration-support.yml Added changelog entry documenting the removal of deprecated streamFunctions configuration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@epugh epugh merged commit a330e81 into apache:main Feb 5, 2026
9 of 11 checks passed
epugh added a commit that referenced this pull request Feb 5, 2026
…4100)

Removes deprecated code from GraphHandler that supported the old streamFunctions configuration method for registering streaming expressions
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.

1 participant