Issue 113 in solrmeter: SolrMeter fails whne testing group search

[solr...@googlecode.com]

Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 113 by ricardo....@gmail.com: SolrMeter fails whne testing group
search
http://code.google.com/p/solrmeter/issues/detail?id=113

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.


--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

[solr...@googlecode.com]

Comment #1 on issue 113 by ricardo....@gmail.com: SolrMeter fails whne

testing group search
http://code.google.com/p/solrmeter/issues/detail?id=113

cHere is a patch I'm using to to stress testing for my app using group
fields.

[solr...@googlecode.com]

Comment #2 on issue 113 by ricardo....@gmail.com: SolrMeter fails whne

testing group search
http://code.google.com/p/solrmeter/issues/detail?id=113

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


Attachments:
patch.diff 2.9 KB

[solr...@googlecode.com]

Comment #3 on issue 113 by tai.dev...@gmail.com: SolrMeter fails whne

testing group search
http://code.google.com/p/solrmeter/issues/detail?id=113

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

[solr...@googlecode.com]

Updates:
Status: Started
Labels: Milestone-Release-0.3.1

Comment #4 on issue 113 by tflo...@gmail.com: SolrMeter fails whne testing
group search
http://code.google.com/p/solrmeter/issues/detail?id=113

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?

Attachments:
solrmeter-113.patch 4.0 KB

[solr...@googlecode.com]

Comment #5 on issue 113 by tflo...@gmail.com: SolrMeter fails whne testing
group search
http://code.google.com/p/solrmeter/issues/detail?id=113

Issue 107 has been merged into this issue.