Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Heartbeating activities should be interruptable when the activities are paused. #602

Open
9 tasks
drewhoskins-temporal opened this issue Feb 20, 2025 · 1 comment
Labels
enhancement New feature or request

Comments

@drewhoskins-temporal
Copy link

Is your feature request related to a problem? Please describe.

When a user runs temporal activity pause, this should work on long-running activities that heartbeat.
Users likely won't have predicted in advance that they're going to need to pause the activity, so ideally this would interrupt the activity's execution by default.

Describe the solution you'd like

Ideally a cancel-like exception could be thrown (in languages where that's appropriate)

  • Users should be able to distinguish between this and cancel, as there maybe different actions they want to take when the activity is ending completely vs when it's going to resume later.

Additional context

Per-SDK Tickets

  • Go -
  • Java -
  • Core -
  • TypeScript -
  • Python -
  • .NET -
  • Ruby -
  • PHP -
  • Temporal CLI -
@drewhoskins-temporal drewhoskins-temporal added the enhancement New feature or request label Feb 20, 2025
@cretz
Copy link
Member

cretz commented Feb 20, 2025

@drewhoskins-temporal - can you confirm whether there is server-side work? RecordActivityTaskHeartbeatResponse.activity_paused is present, just need to confirm if we should always interrupt on that or if interrupt is a separate option the pauser sets and therefore there needs to be another field there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants