[Bug]: Invalid suspension logs a WARN and returns a PENDING result that the service rejects
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 新手友好度
- 48/100
- Issue 类型
- 缺陷
- 描述清晰度
- 基本清楚
- 活跃度
- 活跃
- 技术栈
- aws, java
- 领域
- backend, testing-qa
调研方向
Start with ExecutionManager.preSuspendCheck(), including its callers in deregisterActiveThread and finishCheckpointProcessing, then inspect DurableExecutor's retry path and LocalDurableTestRunner.runUntilComplete. Reproduce case 1 from #752 and trace the PENDING output. Done means invalid suspension is reported with awaited operation details, transient retries remain possible, the no-token path is allowed, and local tests expose invalid output.
由索引模型根据 Issue 内容生成。
描述
Expected Behavior
When the SDK suspends with no pending operation, it returns an output that the service rejects. The SDK should report this state as an SDK error. The error should name the operations that the suspended threads were waiting for.
The invocation should stay retryable. The race in #370 reached this state, and the execution still succeeded on a later invocation.
Actual Behavior
ExecutionManagercallspreSuspendCheck()before it suspends. It does so inderegisterActiveThreadand infinishCheckpointProcessing.preSuspendCheck()looks for a pending operation: aSTEPinPENDING, aWAITorCALLBACKinSTARTED, or aCHAINED_INVOKEinPENDINGorSTARTED.- If it finds none, it logs one WARN line:
Invalid suspension. No operation is pending. The line does not name an operation or a thread. - The SDK then suspends and returns
Status: PENDINGwith no error. The Lambda invocation succeeds, because the handler returns normally. - The service rejects the output with
InvalidParameterValueException: Cannot return PENDING status with no pending operations. - If the cause is deterministic, every replay reaches the same state. The execution fails, and
GetDurableExecutionreportsStatus: FAILEDwith that error.
The customer sees Lambda invocations that succeed, one WARN line per invocation, and a failed execution. None of these names the operation or the SDK defect that caused the failure.
Steps to Reproduce
-
Run case 1 from #752 with
LocalDurableTestRunner. That handler runs afirstSuccessful()parallel withmaxConcurrency(1), then callsget()on the skipped second branch. -
Each invocation returns status
PENDINGwith no error. No operation is pending. -
Each invocation logs:
WARN software.amazon.lambda.durable.execution.ExecutionManager - Invalid suspension. No operation is pending -
LocalDurableTestRunner.runUntilCompletereturnsPENDING. It does not report that the service would reject the output.
SDK Version
2.2.1. Reproduced on main at ef88276 (2.2.2-SNAPSHOT).
Java Version
21
Is this a regression?
Unknown
Additional Context
Three SDK defects have reached this state. Each ended with the service error above.
- #370: a race between checkpoint completion and a waiting thread. Fixed in #659.
- #736: a race in
allCompleted()branch registration. Fixed in #737. - #752:
get()on aSKIPPEDparallel branch. Open.
A fix needs to handle three things.
- Keep the retry. A transient race can clear on a later invocation, as in #370.
DurableExecutoralready rethrows a retryableUnrecoverableDurableExecutionException. Its comment says this is to "let the backend retry the invocation". An error on that path could keep the retry and name the cause. - Allow the no-token path. #706 proposes a
PENDINGreturn when a checkpoint response has no token. That path abandons in-flight operations, so the state can hold no pending operation. A stricter check must not reject that return. - Report it in local tests.
LocalDurableTestRunnerreturnsPENDINGfor this output. It could report the output as invalid, so that a local test shows the outcome the service produces.
A smaller change keeps the current behavior and logs at ERROR with the ID and name of the awaited operation.
- 主要语言
- Java
- 星标
- 28
- 派生
- 13
- 平均合并
- 2 天 3 小时
- 30 天内合并 PR
- 44
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
aws/aws-durable-execution-sdk-java 的其他 Issue
-
documentation pkg:sdk
难度 1/5 1-3 小时 新手友好度 88/100
aws/aws-durable-execution-sdk-java#645 · 1 条评论 ·
维护者通常 1 天内回复
-
enhancement
难度 2/5 1-3 小时 新手友好度 68/100
aws/aws-durable-execution-sdk-java#300 ·
维护者通常 1 天内回复
-
enhancement needs-triage
难度 5/5 一周以上 新手友好度 35/100
aws/aws-durable-execution-sdk-java#779 ·
维护者通常 1 天内回复
-
[Bug]: root handler instrumentation misses the canonical OTel execution context可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭needs-triage
难度 5/5 一周以上 新手友好度 40/100
aws/aws-durable-execution-sdk-java#770 ·
维护者通常 1 天内回复
-
[Feature]: Propagate per-operation trace context for chained invokes可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭enhancement needs-triage
难度 5/5 一周以上 新手友好度 38/100
aws/aws-durable-execution-sdk-java#764 ·
维护者通常 1 天内回复
查看 aws/aws-durable-execution-sdk-java 的全部 Issue
相似的 Issue
-
难度 1/5 1 小时以内 新手友好度 85/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 70/100
commonmark/commonmark-java#460 ·
-
难度 2/5 1-3 小时 新手友好度 72/100
GoogleCloudPlatform/spring-cloud-gcp#4664 ·
维护者通常 1 天内回复
-
enhancement user story
难度 2/5 1-3 小时 新手友好度 70/100
-
难度 2/5 1-3 小时 新手友好度 86/100