Skip to content

refactor(ObjectStoreStorage): light refactoring of writeStream#58898

Draft
joshtrichards wants to merge 2 commits intomasterfrom
jtr/refactor-ObjectStoreStorage-writeStream
Draft

refactor(ObjectStoreStorage): light refactoring of writeStream#58898
joshtrichards wants to merge 2 commits intomasterfrom
jtr/refactor-ObjectStoreStorage-writeStream

Conversation

@joshtrichards
Copy link
Member

  • Resolves: #

Summary

We may want to split this function up into a few focused private methods, but this refactor is just focused on:

  • adding some helpful inline comments to make it easier to follow the logic/flow
  • streamlining logging logic
  • Making sure metadata works with when size is `0

TODO

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Signed-off-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: Josh <josh.t.richards@gmail.com>
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