HttpSolrClient withTheseParamNamesInTheUrl defaults - #4871
Conversation
HttpSolrClient.withTheseParamNamesInTheUrl now has a default set (if you don't call it) that ensures many interesting parameters for routing or fundamental purpose end up in the URL.
dsmiley
left a comment
dsmiley
left a comment
There was a problem hiding this comment.
I like how the tracing test, especially after I revamped it recently, shows the impacts of this change.
It may be tempting to add "command" param for replication but I recall those requests are all GET? No; some are POST so I will add.
|
Gonna merge Sunday if no feedback. One possible consideration is, should these defaults be augmented/merged with a call to explicitly set them. I don't think I care much either way... but a benefit to not doing that is that a user can explicitly insist no params go in the URL if they want. Granted I don't know why they would care. Only one place in Solr, Distributed URP, is this called, which is to set the update distribution params. Update requests don't use any of these default params, so whatever. |
1 participant
HttpSolrClient.withTheseParamNamesInTheUrl now has a default set (if you don't call it) that ensures many interesting parameters for routing or fundamental purpose end up in the URL.