Skip to content

Adding Solr stats to Response - #28

Open
lancew wants to merge 1 commit into
petdance:masterfrom
lancew:master
Open

Adding Solr stats to Response#28
lancew wants to merge 1 commit into
petdance:masterfrom
lancew:master

Conversation

@lancew

@lancew lancew commented Jan 9, 2015

Copy link
Copy Markdown

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
Copy Markdown
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

lancew commented Jan 15, 2015

Copy link
Copy Markdown
Author

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" notifications@github.com 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
Copy Markdown
Owner

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

Adding pod for stats in response
@lancew

lancew commented Jan 21, 2015

Copy link
Copy Markdown
Author

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