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

fix(api-v1): Optimise some SPARQL patterns for Fuseki (DSP-1325) #1814

Merged
merged 2 commits into from Feb 4, 2021

Conversation

benjamingeer
Copy link

resolves DSP-1325

  • Optimise checking for isDeleted, especially where it's likely to help the most.
  • Reorder a few WHERE patterns, especially in extended search.

@benjamingeer benjamingeer self-assigned this Feb 4, 2021
@benjamingeer benjamingeer added API/V1 enhancement improve existing code or new feature bug something isn't working and removed enhancement improve existing code or new feature labels Feb 4, 2021
@benjamingeer benjamingeer changed the title fix(api-v1): Optimise some SPARQL patterns (DSP-1325) fix(api-v1): Optimise some SPARQL patterns for Fuseki (DSP-1325) Feb 4, 2021
Copy link
Contributor

@SepidehAlassi SepidehAlassi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works pretty fast, thanks!

@subotic
Copy link
Collaborator

subotic commented Feb 4, 2021

@benjamingeer @SepidehAlassi can we merge?

@benjamingeer benjamingeer merged commit 4edc27c into main Feb 4, 2021
@benjamingeer benjamingeer deleted the wip/DSP-1325-v1-search branch February 4, 2021 18:46
@subotic
Copy link
Collaborator

subotic commented Feb 4, 2021

Thanks :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API/V1 bug something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants