Repository navigation
feat: add sharded frame-tree analysis and lightweight cache reuse #269
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
Merged
Merged
Changes from all commits
Commits
Show all changes
8 commits
Select commit
Hold shift + click to select a range
331a06e
feat: add sharded frame-tree analysis and lightweight cache reuse
shaanjiangcun f0309a4
fix: cancel superseded frame-tree analysis cooperatively
shaanjiangcun da53b58
fix: cancel frame trees when analyzer caches are invalidated
shaanjiangcun 39d0793
fix: address frame evidence and sharded acceptance review findings
shaanjiangcun 7bed620
fix: include frame-tree queue waits in query deadlines
shaanjiangcun 1cab969
docs: clarify lightweight compatible cache reuse checks
shaanjiangcun 54806d1
merge: integrate latest upstream main and reconcile bilingual guides
shaanjiangcun 6429b76
test: adapt upstream cache coverage to lightweight validation
shaanjiangcun File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[P2] 将 v3 联动目标与旧 v2 的 GUI 操作入口明确分开
这段正确说明历史结果不验证 v3,但本页 L7 仍写唯一目标是
101f65b,后续步骤仍要求使用该历史 GUI 构建;协议指南 L489–494 和 quickstart 也继续承诺这个入口。固定101f65b的 parser 使用 v2 salt,hash 不匹配会进入重新解析,并删除旧 manifest/log 后启动 producer(固定源码,L43、L189–202、L243–265)。按当前整段说明把新 v3 产物交给这个旧消费者,不是简单的“尚未验收”,而是不能复用且可能改写缓存。这里不要求回退已说明的 v3 联动方向。请将
101f65b/targetRevision明确标为历史 schema/身份参考,并给出配套 v3 consumer 的固定 revision 与匹配 hash/member 语义;在其建立和验收前不要继续承诺旧入口支持新导出。中英文同步说明使用独立可信 pickle 副本与相邻新缓存验收,避免旧 GUI 接触唯一产物。There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已在 7bed620 同步中英文 README、quickstart、协议指南和互操作验收文档。保留 parser v3,将 101f65b 和 targetRevision 明确标为历史 v2 schema/身份参考,旧 GUI 操作步骤单列为历史流程,不再承诺它支持当前 v3 导出;同时说明旧 GUI 可能重新解析并改写缓存。
目前尚无已核验的配套 v3 consumer 固定 revision/build,文档如实标记未建立该证据,未编造版本或宣称 GUI 复用通过。新增当前 v3 验收要求:消费端匹配 v3 salted cacheHash 和空 member 值的结构清单语义,固定消费端 revision/build,并使用独立可信 pickle 副本与新建邻接缓存,验证实际复用及无意外修改后再宣称支持。文档治理 27 项测试通过;本次未运行 GUI,也未修改 Insight 仓。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已对固定提交
7bed6208b6dfb72eb0e7ead89dc3d7e15e5cfa6c的中英文 README、quickstart、协议和验收说明做静态复审及独立交叉检查。原 P2 的旧 v2 GUI 支持承诺已修正:101f65b/targetRevision已明确为历史 schema/身份参考,旧 GUI 重解析改写风险及独立可信副本、新邻接缓存的验收要求均已写明,不再把历史验收当作 v3 证明。当前仍缺已核验的配套 v3 consumer 固定 revision/build 和实际 GUI 复用证据;消费端需同时匹配 v3 salted
cacheHash与空 member 值的结构清单语义。本轮未运行 GUI或测试,也未据此确认联合兼容。另有一项低优先级文档清理,不维持或新增上述 P2:英文 quickstart:144–146 / 中文:118–119 仍称全部成员哈希必须匹配。现行
snapshot_cache.py:248–268只比较成员清单的键,不比较摘要值;结构合法的内容变更可被复用。建议把这两处改为源身份、成员清单和结构检查,并说明不保证发现任意行内容修改,与协议页一致。There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已静态复审并独立交叉核对
1cab969ef6fb958ac11518df2b04074b8019cbed:该提交仅修改中英文 quickstart/database 四份文档,运行时和测试未变。此前非阻塞的成员哈希表述已修正为成员清单及结构检查,明确不比较摘要值、不保证发现任意内容编辑,metadata/复用成功不代表内容完整性;与现行实现一致。本次增量未发现新的实质问题。本轮未运行测试、构建或 GUI。当前 head 仍与 main 有冲突且无 CI 运行;配套 v3 consumer 固定 revision/build 与实际 GUI 复用验收仍待补齐,文档修正不替代这些验收。