Skip to content

Reduce attachment upload memory pressure#27

Merged
vincentkoc merged 2 commits intomainfrom
vincentkoc-code/opik-attachment-upload-audit
Mar 6, 2026
Merged

Reduce attachment upload memory pressure#27
vincentkoc merged 2 commits intomainfrom
vincentkoc-code/opik-attachment-upload-audit

Conversation

@vincentkoc
Copy link
Copy Markdown
Member

Details

  • avoid loading the full attachment into memory with readFile() before upload
  • switch to openAsBlob() and slice blob parts for multipart uploads
  • add multipart upload regression coverage so request bodies stay bounded by part size

Change checklist

  • User facing
  • Documentation updated (if needed)
  • Tests added/updated (if needed)
  • Breaking changes documented (if any)

Issues

  • Resolves #
  • OPIK-

Testing

  • npm run lint
  • npm run test -- src/service/attachment-uploader.test.ts
  • npm run smoke

Documentation

  • None

@vincentkoc vincentkoc marked this pull request as ready for review March 6, 2026 04:36
@vincentkoc vincentkoc merged commit 98f77c0 into main Mar 6, 2026
1 check passed
@vincentkoc vincentkoc deleted the vincentkoc-code/opik-attachment-upload-audit branch March 6, 2026 04:39
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