search

AnshumanDwivediTCS / solrmeter

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

SolrMeter fails whne testing group search #113

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. Create a group query (group=true&group.field=...)
2. Click play

What is the expected output? What do you see instead?
I'm seeing NullPointers in the log

What version of the product are you using? On what operating system?
0.3.1 on OS X

Please provide any additional information below.

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

GoogleCodeExporter commented 9 years ago 
[deleted comment]
GoogleCodeExporter commented 9 years ago 
Here is a patch I'm using to do stress testing for my app using field 
collapsing/grouping.

Original comment by ricardo....@gmail.com on 3 Apr 2013 at 9:38

Attachments:

GoogleCodeExporter commented 9 years ago 
The patch works perfectly for me, and this is very much appreciated. I spotted 
the issue when testing with groups last week, and was prepared to write a patch 
myself, but you've removed that need! There is a duplicate issue logged in this 
project, which I'll comment on in a moment, in order to redirect them here.

Note to committers : Can we get this merged into the trunk please, and a new 
build created? 

On a related note have you thought about moving this project to 
GitHub/Bitbucket? This will make it much easier to contribute to (which I would 
be keen to do).

Best wishes,

Daniel

Original comment by tai.dev...@gmail.com on 14 Apr 2013 at 7:23

GoogleCodeExporter commented 9 years ago 
Thanks for the patch. The problem with this fix is that it requires 
group.ngroup=true, which is not the default. I think we should use the matches 
instead (count the number of docs, not the number of groups).
Also, I don't think we should sum the matches of the different group commands. 
Matches should be the same in all commands (Solr returns one command per 
grouping field/func/query, not per group)
Would you mind testing this patch and see if it works for you?

Original comment by tflo...@gmail.com on 22 May 2013 at 12:57

Attachments:

GoogleCodeExporter commented 9 years ago 
Issue 107 has been merged into this issue.

Original comment by tflo...@gmail.com on 22 May 2013 at 12:58