SOLR-17715: Remove qt parameter from the final parameters sent to server

[r4mercur]

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

Description

Please provide a short description of the changes you're making with this pull request.

Solution

Please provide a short description of the approach taken to implement your solution.

Tests

Please describe the tests you've developed or run to confirm this patch implements the feature or solves the problem.

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

[dsmiley]

Coming back to this, I think this is close to what we should do. But I'd rather see the path extracted in the constructor so that QueryRequest no longer overrides getPath.

Also the placement of these tests seems arbitrary/wrong to me, but the tests themselves are very good. Can you please create a new PR? If you are no longer interested, I'll maybe tell an AI agent to do this.

[epugh]

I was just updating a downstream plugin for Solr and it had qt and it would be so nice to get rid of it!

[dsmiley]

And add a new constructor to QueryRequest to put the path first explicitly without resorting to "qt" param. The others can be deprecated.

[r4mercur]

@dsmiley one question before i would start a new try to tackle this. Where would you see the tests-cases best placed ?

[dsmiley]

Just start a new testcase, as I see no existing one.

Thanks for your interest!