SolrMeter fails whne testing group search

[Google Code Exporter]

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 [e-mail] on 3 Apr 2013 at 9:35

[Google Code Exporter]

[deleted comment]

[Google Code Exporter]

Here is a patch I'm using to do stress testing for my app using field 
collapsing/grouping.

Original comment by [e-mail] on 3 Apr 2013 at 9:38

Attachments:

• patch.diff

[Google Code Exporter]

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 [e-mail] on 14 Apr 2013 at 7:23

[Google Code Exporter]

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 [e-mail] on 22 May 2013 at 12:57

• Changed state: Started
• Added labels: Milestone-Release-0.3.1

Attachments:

• solrmeter-113.patch

[Google Code Exporter]

Issue 107 has been merged into this issue.

Original comment by [e-mail] on 22 May 2013 at 12:58