Closed RobinTF closed 3 weeks ago
Attention: Patch coverage is 91.83673%
with 16 lines
in your changes missing coverage. Please review.
Project coverage is 89.11%. Comparing base (
05a3f69
) to head (b1014d1
). Report is 3 commits behind head on master.
Files with missing lines | Patch % | Lines |
---|---|---|
src/engine/TransitivePathImpl.h | 91.45% | 1 Missing and 9 partials :warning: |
src/engine/TransitivePathBase.cpp | 88.46% | 0 Missing and 6 partials :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
No test result changes.
Issues
15 New issues
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
This PR enables the
TransitivePath
operation to yield its result lazily and to consume its left/right child lazily. Note that the graph which is transitively traversed needs to be fully materialized due to the underlying algorithm. E.G when computing the (large) result ofwdt:P31/wdt:P279*
, the large result and thewdt:P31
can be dealt with lazily, but the fullwdt:P279
predicate needs to be materialized.