Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding Solr stats to Response #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

lancew
Copy link

@lancew lancew commented Jan 9, 2015

This is a small addition to return in the response the stats if you have asked for them with &stats=true&stats.field=foo.

@petdance
Copy link
Owner

Thanks for the changes.

Before I can merge this, the docs need to be updated to tell the user how to use this feature, or at the very least that the feature exists.

@lancew
Copy link
Author

lancew commented Jan 15, 2015

Ok I'll add something to explain the request that gives you the stats.
Where would you prefer it goes?
On 15 Jan 2015 14:38, "Andy Lester" [email protected] wrote:

Thanks for the changes.

Before I can merge this, the docs need to be updated to tell the user how
to use this feature, or at the very least that the feature exists.


Reply to this email directly or view it on GitHub
#28 (comment).

@petdance
Copy link
Owner

Somewhere in W::S::Response, I'd think. There's no accessor for it?

Adding pod for stats in response
@lancew
Copy link
Author

lancew commented Jan 21, 2015

I have now added a little pod for the stats in response (I think).
I shall take a look at how to put requesting stats into the query side also for another pull request at a later date.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants