Closed RobinTF closed 2 days ago
No test result changes.
Attention: Patch coverage is 92.30769%
with 1 line
in your changes missing coverage. Please review.
Project coverage is 89.28%. Comparing base (
39ca684
) to head (cb06fa3
). Report is 4 commits behind head on master.
Files with missing lines | Patch % | Lines |
---|---|---|
src/engine/TransitivePathImpl.h | 87.50% | 0 Missing and 1 partial :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
🚨 Try these New Features:
Issues
1 New issue
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
The introduction of lazy transitive paths via #1595 caused transitive path operations with lazy inputs and a fully materialized result to store the same local vocab once per input row. This lead to OOM errors even if that local vocab was empty. This commit fixes that behavior by
Note that this still might lead to redundant stores in the local vocab, but deduplicating those will be done in a separate PR.