Repository navigation
Conversation
Signed-off-by: Iris3971 <293864676+Iris3971@users.noreply.github.com>
Signed-off-by: Iris3971 <293864676+Iris3971@users.noreply.github.com>
Signed-off-by: Iris3971 <293864676+Iris3971@users.noreply.github.com>
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.
改了什么
Fixes #137
修复 History Import 在分块处理失败后仍推进续传状态,导致 Pause 后 Resume 时跳过失败分块的问题。
新增
failed_chunk_indices持久化记录失败分块。Resume 时会优先补跑历史失败分块,再继续处理主游标之后的新分块;补跑成功后同步清理失败状态。为什么这么改
没有直接改成“分块失败就停止推进
processed”,因为当前 History Import 的既有行为允许长批次在单个 chunk 失败后继续处理后续内容,并最终以partial状态结束。因此这里保留现有主游标继续向后的行为,同时单独记录失败分块:
processed已推进而静默跳过失败内容;chunks_failed;partial后重新导入的行为。该问题使用合成输入和 mock extraction 独立复现,并确认在 main、3.8.0 testing 以及当前 testing 基线上均存在。
测过没有
已跑针对 History Import 的专项验证:
回归测试覆盖:
chunks_failed;failed_chunk_indices、chunks_failed与 terminal status 保持一致。仓库普通 Tests workflow 仍会在既有 Bandit security check 处失败;当前 upstream testing 基线本身也存在同一 security-check failure,因此该红灯不由本次修改引入。
Signed-off-by:(git commit -s),表示我有权提交这些代码 —— 见 [DCO](../DCO) 与 [CONTRIBUTING.md](../CONTRIBUTING.md)