Skip to content

Conversation

dandavison
Copy link
Contributor

@dandavison dandavison commented Apr 2, 2025

Initial Python Temporal Nexus implementation.

Temporal SDK PR to accompany

Contains Nexus worker, components for users to define workflow-backed Nexus operations, and the ability to start and cancel a Nexus operation from a workflow.

Notes for reviewers

  • nexusrpc.handler.start_workflow is a top-level static function, but currently there's still a public contextvar object nexusrpc.handler.temporal_operation_context. Let's settle on approach there. If we're using module-level getters I think they need to be named something like get_client() etc, despite the fact that we have activity.metric_meter() ? Little bit more discussion needed there.
  • I'll do another cleanup pass on docstrings, API docs, etc.

@dandavison dandavison force-pushed the nexus branch 2 times, most recently from a845a6f to 7a121bb Compare April 5, 2025 15:08
@dandavison dandavison force-pushed the nexus branch 10 times, most recently from 862e4f9 to 8ac0192 Compare April 17, 2025 12:17
@dandavison dandavison force-pushed the nexus branch 4 times, most recently from 8940d51 to 520aecf Compare April 26, 2025 16:16
@dandavison dandavison force-pushed the nexus branch 2 times, most recently from 2fc971f to 8bd6011 Compare April 29, 2025 13:11
@dandavison dandavison changed the title Nexus prototype Nexus May 8, 2025
@dandavison dandavison force-pushed the nexus branch 2 times, most recently from 5d54f48 to 8ddfb94 Compare May 24, 2025 01:49
@dandavison dandavison force-pushed the nexus branch 7 times, most recently from fa3e8ec to f7bf47b Compare May 27, 2025 20:34
@dandavison dandavison force-pushed the nexus branch 2 times, most recently from 12f4d49 to 9a10b13 Compare July 8, 2025 03:15
@dandavison dandavison merged commit 4165fc5 into main Jul 8, 2025
16 checks passed
@dandavison dandavison deleted the nexus branch July 8, 2025 04:03
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.

3 participants