[ https://issues.apache.org/jira/browse/SOLR-4143?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13533465#comment-13533465 ]
Shawn Heisey commented on SOLR-4143: ------------------------------------ I did look into the idea of making it so that when an alternate parameter is used to say "just change the path, don't include the parameter." The problem with that is that at the point where I'd have to remove the QT parameter from the request, the parameters are in a SolrParams instead of a ModifiableSolrParams, so I can't remove a parameter. It would probably be pretty easy to change that, but I figure there might be a particular reason the read-only object was chosen. Can anyone comment on that? > setRequestHandler - option to not set qt parameter > -------------------------------------------------- > > Key: SOLR-4143 > URL: https://issues.apache.org/jira/browse/SOLR-4143 > Project: Solr > Issue Type: Improvement > Components: clients - java > Affects Versions: 4.0 > Environment: solr-impl 4.1-SNAPSHOT 1416639M - ncindex - > 2012-12-03 12:54:38 > Reporter: Shawn Heisey > Fix For: 4.1, 5.0 > > Attachments: SOLR-4143.patch, SOLR-4143.patch > > > The setRequestHandler method does what I expect in one way - it changes the > URL path from /select to the String argument. It is however doing something > that I did not expect, which is setting the qt parameter on the query as > well. Here is the code: > private static final String PING_HANDLER = "/admin/ping"; > query.setRequestHandler(PING_HANDLER); > This is resulting in the following exception being logged in my Solr 3.5.0 > servers. I am not including the whole exception, because this issue is for > SolrJ 4, not Solr 3.5, and the 3.5 version is working as it was designed. > {code} > Dec 3, 2012 4:04:01 PM org.apache.solr.common.SolrException log > SEVERE: org.apache.solr.common.SolrException: Cannot execute the > PingRequestHandler recursively > at > org.apache.solr.handler.PingRequestHandler.handleRequestBody(PingRequestHandler.java:60) > at > org.apache.solr.handler.RequestHandlerBase.handleRequest(RequestHandlerBase.java:208) > {code} > I believe it would be useful to include an alternate setRequestHandler method > that includes a boolean option deciding whether or not to also set the qt > parameter. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org