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

feat: Split words of query and send 1 query per word #686

Closed
wants to merge 3 commits into from

Conversation

ptbrowne
Copy link
Collaborator

@ptbrowne ptbrowne commented Sep 7, 2022

This PR tries to improve the search times when there are several words.
Instead of considering the full query, we send several query for each word, ignoring very small words that are not capitalized.

  • feat: Split words of query and send 1 query per word
  • feat: Default debug search uses drafts
  • fix: Ignore search terms that are not capitalized and under 2 letters of length

This is related to #683

@vercel
Copy link

vercel bot commented Sep 7, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
visualization-tool ✅ Ready (Inspect) Visit Preview Sep 7, 2022 at 4:49PM (UTC)

@ptbrowne
Copy link
Collaborator Author

ptbrowne commented Sep 7, 2022

@l00mi When you have further information on whether the stardog full text search or our queries can be improved, could you post it here ? Thanks 🙇

@l00mi
Copy link

l00mi commented Sep 7, 2022

Yes will do. Thank you for this inputs.

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

Successfully merging this pull request may close these issues.

None yet

2 participants