Skip to content

Conversation

@gus-asf
Copy link
Contributor

@gus-asf gus-asf commented Jan 2, 2026

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

The operations in this filter are expected to be required for proper functioning of Solr, and it should not contain anything that could reasonably be removed, or replaced with an alternate configuration.

For the moment this is replacing MDC filter since we seem to have actually made a few portions of the solr code dependent on it. Depending on how SOLR-18050 is resolved we might re-extract MDC from the rest of this.

… wrapping operations required for proper functioning of Solr
@gus-asf
Copy link
Contributor Author

gus-asf commented Jan 2, 2026

Hmm This change essentially consumes the changes from SOLR-18042, but the build is complaining when I remove the (now inaccurate/misleading) entry for SOLR-18042 ... What's the intended method to handle something that changes twice before a release?

@janhoy
Copy link
Contributor

janhoy commented Jan 2, 2026

Hmm This change essentially consumes the changes from SOLR-18042, but the build is complaining when I remove the (now inaccurate/misleading) entry for SOLR-18042 ... What's the intended method to handle something that changes twice before a release?

The changelog checker script does not anticipate a removal. Should probably allow the case where a yml is deleted. I'll prepare a fix.

For this PR you can simply ignore the failure and merge :)

@janhoy
Copy link
Contributor

janhoy commented Jan 2, 2026

See #3999 for proposed change...

@@ -0,0 +1,8 @@
# See https://github.com/apache/solr/blob/main/dev-docs/changelog.adoc
title: SOLR 18044 - EssentialSolrRequestFilter has been added to perform non-optional request wrapping operations required for proper functioning of Solr
Copy link
Contributor

Choose a reason for hiding this comment

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

Please don't include JIRA number in Title, it is redundant with the links below which will anyway be rendered into CHANGES.md automatically

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah ok, it arrived via the generation so I left it. Will fix

@gus-asf gus-asf requested a review from dsmiley January 2, 2026 16:35
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.

2 participants