From f12d13826ed629c4c2291f2635fb58443d6670b2 Mon Sep 17 00:00:00 2001 From: DABH Date: Fri, 13 Nov 2026 19:52:59 -0600 Subject: [PATCH 1/2] Reword the reason for the Arize sample's temporalio pin The pyproject comment and the README's reset note now state only that 1.36.0 applies Workflow Task jobs as a single batch and that later releases do not do so by default, without calling the pin temporary or promising that a later release restores the behavior. --- arize_tracing/pyproject.toml | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/arize_tracing/pyproject.toml b/arize_tracing/pyproject.toml index 8ac1453c..02a9290f 100644 --- a/arize_tracing/pyproject.toml +++ b/arize_tracing/pyproject.toml @@ -3,10 +3,9 @@ name = "temporalio-samples-arize-tracing" version = "0.1.0" requires-python = ">=3.11" dependencies = [ - # Pinned: 1.36.0 applies workflow activation jobs as a single batch, which - # keeps a reset's reapplied update from reusing a span ID of the original - # run. 1.36.1 (temporarily?) undoes that, so we're pinning to 1.36.0 - # explicitly until a future release resolves this issue one way or another. + # 1.36.0 applies workflow activation jobs as a single batch, which keeps a + # reset's reapplied update from reusing a span ID of the original run. + # Later releases do not do this by default (temporalio/sdk-python#2121). "temporalio[opentelemetry]==1.36.0", # 1.1.0 fixes span parenting when one worker runs both workflows and # activities; its openinference extra brings the Agents SDK instrumentation. From ff6bac5ac71525b960c4ee33ed8e5600a494f27a Mon Sep 17 00:00:00 2001 From: DABH Date: Fri, 13 Nov 2026 19:54:12 -0600 Subject: [PATCH 2/2] Drop "currently" from the README's reset note Same feedback as the pyproject comment: no temporary status. --- arize_tracing/README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arize_tracing/README.md b/arize_tracing/README.md index 97d96ba5..2d0c94ff 100644 --- a/arize_tracing/README.md +++ b/arize_tracing/README.md @@ -168,7 +168,7 @@ temporal workflow reset --workflow-id --type FirstWorkflowTask --r uv run verify_trace.py --workflow-id --expect-runs 2 ``` -The reset case currently needs temporalio 1.36.0 exactly (see PR #420). +The reset case needs temporalio 1.36.0 exactly (see PR #420). ![Activity retry attempts in Phoenix](phoenix-retry-attempts.png)