perf: When property path is bound both sides, use both bindings

Callidon / sparql-engine

🚂 A framework for building SPARQL query engines in Javascript/Typescript

https://callidon.github.io/sparql-engine

MIT License

97 stars 14 forks source link

perf: When property path is bound both sides, use both bindings #67

Closed sroze closed 3 years ago

sroze commented 3 years ago

At the moment, even if both sides of the property path are bound, the triple match will have one side unbound: this is a performance bottleneck, especially for large datasets.

This PR changes this.

Callidon commented 3 years ago

Thanks for the fix, I'm merging it right away.

Callidon commented 3 years ago

@sroze Do you have others optimizations/bug fixes in preparation for the package? Just to know if I can publish a minor release right away, to publish your contributions, or if I need to wait for more stuff to come 😃

sroze commented 3 years ago

@Callidon so far so good, that's all I found, you can release a minor version. Thank you!