perf!: cache compact DAG analysis - #180
Merged
Merged
Conversation
eitsupi
marked this pull request as ready for review
August 29, 2026 15:00
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Breaking change
The public ManifestGraphCache API is replaced by ManifestAnalysisCache and the non-exhaustive ManifestAnalysis value. dlin is pre-1.0 and there are no known external users of the old raw cache API.
Benchmark
Medium cache workload, 512 models, release binary, 10 runs / 2 warmups. Persistent state is controlled; OS page cache is not flushed.
The final warm median is about 51% lower than the same-main baseline. The cache file grows from 175,969 bytes to 188,679 bytes (about 7.2%) without persisting the raw typed Manifest. SQL, manifest, and column cold/warm/no-cache semantic probes all pass, and warm cache files remain unchanged.
Validation