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

Implemented ability for sorting and counting on solr side via 'direct index query" using indexQueryBuilder() #1182

Open
wants to merge 2 commits into
base: titan05
Choose a base branch
from

Conversation

dmytro-babichev
Copy link

We faced with performance issue on our project when sorting and counting hundreds of thousands of vertices by different vertex properties and existance of relationships between them.

To solve the issue we decided to denormalize our data, index all sorting and filtering properties in solr and use 'direct index queries'. However we didn't find a way, how to use those indices for sorting via indexQuery(). Also it was impossible to get 'response.numFound' from solr results to return total number of results to user.

My branch contains required changes, which allow us to sort blazingly fast on the solr side via titan. We are able to return total number of results satisfying the search query as well.

@titan-cla
Copy link

Hi @laory, thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes.

@titan-cla
Copy link

You did it @laory!

Thank you for signing the Contribution License Agreement.

@dmytro-babichev dmytro-babichev changed the title Implemented ability for sorting on solr side via 'direct index query"… Implemented ability for sorting and counting on solr side via 'direct index query" using indexQueryBuilder() Nov 12, 2015
@dalaro dalaro added this to the Titan1.1 milestone Dec 17, 2015
@dalaro dalaro self-assigned this Dec 17, 2015
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.

3 participants