renatoh commented on PR #4568: URL: https://github.com/apache/solr/pull/4568#issuecomment-4851185342
> While I like the fix relating to minShouldMatch, as it's non-released... I'm hesitant to approve the `clauses()` as this is technically a change to existing behavior, even if I'm somewhat suspicious someone depends on this behavior. It's really unfortunate Solr doesn't have a consistent/unified approach to parameter resolution that all users can understand. Instead, each QP makes its own decision on how each param is resolved. :-( I see you point. I changed it because what I got from one of your earlier comments is that BoolQParser should read only localParams, and not params. Such a change is only possible in a major version since it's technically breaking backward compatibility? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
