Skip to content

docs: add ADR for filesystem payload offloading#512

Open
zhongkechen wants to merge 3 commits into
mainfrom
codex/adr-filesystem-payload-offloading
Open

docs: add ADR for filesystem payload offloading#512
zhongkechen wants to merge 3 commits into
mainfrom
codex/adr-filesystem-payload-offloading

Conversation

@zhongkechen

@zhongkechen zhongkechen commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add ADR-005 for filesystem payload offloading design
  • compare reusing SerDes with introducing a PayloadOffloader interface
  • document extra package naming, caching, threading, exception handling, and AI recommendation

Preview

https://github.com/aws/aws-durable-execution-sdk-java/blob/codex/adr-filesystem-payload-offloading/docs/adr/005-filesystem-serdes.md

Issue

Testing

  • not run; docs-only change

@zhongkechen zhongkechen requested a review from a team July 2, 2026 21:35
@zhongkechen zhongkechen linked an issue Jul 2, 2026 that may be closed by this pull request
@zhongkechen zhongkechen removed a link to an issue Jul 2, 2026
@zhongkechen zhongkechen self-assigned this Jul 2, 2026
@zhongkechen zhongkechen changed the title Add ADR for filesystem payload offloading docs: add ADR for filesystem payload offloading Jul 2, 2026

@SilanHe SilanHe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like that we're discussing this decision in open source.

So are you proposing Approach B : Create a PayloadOffloader Interface as is recommended by the AI?

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.

2 participants