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

Support pagination with Solr Result Grouping #145

Open
wants to merge 1 commit into
base: master
from

Conversation

@mpdude
Copy link

commented Jun 5, 2016

When using result grouping, we don't get a simple number of documents matched. Instead, related documents are collapsed into groups and the groups are what we (probably) want to paginate.

This is a bit whacky as it is technically possible to specify different grouping criteria. In that case, we'd get multiple lists of groups back.

This change only looks at the first grouping applied as I don't see how we could make sense of the multi-group case.

Support pagination with result grouping
When using [result grouping](https://cwiki.apache.org/confluence/display/solr/Result+Grouping), we don't get a simple number of documents matched. Instead, related documents are collapsed into groups and the groups are what we (probably) want to paginate.

This is a bit whacky as it is technically possible to specify different grouping criteria. In that case, we'd get multiple lists of groups back. 

This change only looks at the first grouping applied as I don't see how we could make sense of the multi-group case.
@mpdude

This comment has been minimized.

Copy link
Author

commented Jun 5, 2016

/cc @dword123

@NiR-

This comment has been minimized.

Copy link
Member

commented Jun 2, 2017

Thank you for your contribution. Could you add a test case for result grouping please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.