Skip to content

Pin the Arize sample to temporalio 1.36.0 - #420

Merged
DABH merged 3 commits into
mainfrom
arize-pin-1360
Nov 14, 2026
Merged

DABH merged 3 commits into
mainfrom
arize-pin-1360

Conversation

@DABH

@DABH DABH commented Nov 14, 2026

Copy link
Copy Markdown
Contributor

This PR pins the Arize sample to temporalio==1.36.0.

The sample's reset case depends on 1.36.0 applying a Workflow Task's jobs as a single batch, so a reapplied Update's span gets a fresh ID instead of reusing the original run's. temporalio/sdk-python#2121 turns that off by default again for 1.36.1 while the cancellation semantics are settled in temporalio/sdk-python#2124, so the next release would bring the duplicate span IDs back. The pyproject comment and the README's reset note say why the pin exists and when to lift it. The lock resolution is unchanged; the sample's tests, ruff, and mypy pass.

The reset case depends on 1.36.0 applying a Workflow Task's jobs as a
single batch, so a reapplied update's span gets a fresh ID instead of
reusing the original run's. temporalio/sdk-python#2121 turns that off by
default again for 1.36.1 while the cancellation semantics are settled in
temporalio/sdk-python#2124, so the next release would bring the
duplicate IDs back. The pyproject comment and the README say why the pin
exists and when to lift it.
Comment on lines +8 to +9
# 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.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# 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.
# run.

@DABH
DABH merged commit de5a968 into main Nov 14, 2026
14 checks passed
@DABH
DABH deleted the arize-pin-1360 branch November 14, 2026 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants