docs: add encyclopedia page for external storage#4380
docs: add encyclopedia page for external storage#4380lennessyy merged 3 commits intolarge-payload-prereleasefrom
Conversation
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📖 Docs PR preview links |
drewhoskins-temporal
left a comment
drewhoskins-temporal
left a comment
There was a problem hiding this comment.
This is great, thanks for all the improvements.
A few final things I noticed.
| payloads from external storage in parallel, then passes them back through the Payload Codec and Payload Converter to | ||
| reconstruct the original data. | ||
|
|
||
| The SDK parallelizes uploads and downloads to minimize latency. When a single Workflow Task involves multiple payloads |
There was a problem hiding this comment.
nit: felt like you repeated yourself in this paragraph.
There was a problem hiding this comment.
The second sentence explains what parallelization means for this context. Not every reader will know what "parallelizes uploads and downloads" implies practically, so the follow-up makes it concrete. And if they do know, they can read the first sentence and skim or skip the rest. Happy to tighten the wording if it feels redundant, but I'd keep the explanation itself.
| If the external store is temporarily unreachable during an upload or download, the operation fails the Workflow Task. | ||
| Temporal automatically retries failed Workflow Tasks, so the operation resumes when the store becomes available again. |
There was a problem hiding this comment.
This is only true when the storage is uploaded from a workflow task. Payloads can come from users' clients, outside the workflow. Maybe just cut this for now?
Co-authored-by: Drew Hoskins <drew.hoskins@temporal.io>
2 participants
Summary
This is PR 2 of 3, split from #4333.
Test plan
🤖 Generated with Claude Code
┆Attachments: EDU-6150 docs: add encyclopedia page for external storage