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

[PJRT:CPU] Implement PjRtDevice::PoisonExecution() API. #23260

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

copybara-service[bot]
Copy link

@copybara-service copybara-service bot commented Feb 28, 2025

[PJRT:CPU] Implement PjRtDevice::PoisonExecution() API.

PjRtDevice::PoisonExecution() API provides a mechanism to mark output buffers
of a pending asynchronous execution with an error, where the error is
constructed above PjRt and potentially reflect a broader view on the system
state. This change implements this API for the PJRT CPU client to improve
support for cancelling pending executions. The state of poisonable executions
is owned by TfrtCpuDevice and accessed from TfrtCpuExecutable as via a new
class TfrtCpuAsyncExecutionTracker.

@copybara-service copybara-service bot force-pushed the test_732212487 branch 9 times, most recently from f8d16c0 to 4f77bcf Compare March 2, 2025 05:13
`PjRtDevice::PoisonExecution()` API provides a mechanism to mark output buffers
of a pending asynchronous execution with an error, where the error is
constructed above PjRt and potentially reflect a broader view on the system
state. This change implements this API for the PJRT CPU client to improve
support for cancelling pending executions. The state of poisonable executions
is owned by `TfrtCpuDevice` and accessed from `TfrtCpuExecutable` as via a new
class `TfrtCpuAsyncExecutionTracker`.

PiperOrigin-RevId: 732212487
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.

1 participant