techact / solrmeter

Automatically exported from code.google.com/p/solrmeter
0 stars 0 forks source link

Parameter FL can't be overwritten with extra parameters #114

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Set a custom FL parameters int extra parameter list
2. Run a load test
3. Check the output, SolrMeter will add two FL: *,score and the one you defined.

What is the expected output? What do you see instead?

I would expect to see my field list plus the score if not already there. I see 
both

What version of the product are you using? On what operating system?

0.3 on OS X

Please provide any additional information below.

The following small patch fixes the issue

--- 
solrmeter/src/main/java/com/plugtree/solrmeter/model/generator/ComplexQueryGener
ator.java   (revision 316)
+++ 
solrmeter/src/main/java/com/plugtree/solrmeter/model/generator/ComplexQueryGener
ator.java   (working copy)
@@ -107,8 +107,8 @@
     query.setQuery(queryExtractor.getRandomQuery());
     //            query.setQueryType(executor.getQueryType());
     query.setQueryType(queryType);
+    this.addExtraParameters(query);
     query.setIncludeScore(true);
-    this.addExtraParameters(query);

     if(useFacets) {
       addFacetParameters(query);

Original issue reported on code.google.com by ricardo....@gmail.com on 8 Apr 2013 at 3:07

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r317.

Original comment by tflo...@gmail.com on 22 May 2013 at 11:29

GoogleCodeExporter commented 9 years ago

Original comment by tflo...@gmail.com on 22 May 2013 at 11:30

GoogleCodeExporter commented 9 years ago

Original comment by tflo...@gmail.com on 22 May 2013 at 11:31