Skip to content

Fix workflow query timeout client reuse#9

Merged
rmcdaniel merged 1 commit into
mainfrom
fix/query-timeout-client-cache
May 16, 2026
Merged

Fix workflow query timeout client reuse#9
rmcdaniel merged 1 commit into
mainfrom
fix/query-timeout-client-cache

Conversation

@rmcdaniel
Copy link
Copy Markdown
Member

@rmcdaniel rmcdaniel commented May 15, 2026

Closes #8.

Summary

  • use a fresh short-timeout client for the cluster-info probe so the real workflow query request can build its own longer-timeout client
  • keep the shared cached client path for the actual command execution instead of poisoning it with the 5-second probe client
  • update the regression test to exercise the real cached-client path that failed in production

Validation

  • composer install --no-interaction --prefer-dist
  • php ./vendor/bin/phpunit tests/Commands/WorkflowHistoryQueryTest.php tests/Commands/MutatingCommandJsonTest.php
  • manual JSON workflow query smoke against a local Durable Workflow server

@rmcdaniel rmcdaniel merged commit 5c7fe0e into main May 16, 2026
2 checks passed
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.

workflow:query reuses 5s cluster-info client timeout and times out against Python workflows

2 participants