storj / uplink

Storj network Go library
MIT License
115 stars 18 forks source link

Memory usage with large uploads on large bandwidth links #148

Closed shaupt131 closed 1 month ago

shaupt131 commented 1 year ago

Background:

The community has reported memory usage and speed issues with the latest uplink release (containing refactored upload codepath). See forum posts here: https://forum.storj.io/t/upload-refactoring/22559/1. Discussion starts 3 or 4 posts down from OP.

Acceptance Criteria:

-Completed investigation into the reported issues. -If low effort (2 or less) implement a fix under this ticket. If not, create follow-up ticket(s) accordingly.

storjrobot commented 1 year ago

This issue has been mentioned on Storj Community Forum (official). There might be relevant details there:

https://forum.storj.io/t/upload-refactoring/22559/20

iglesiasbrandon commented 1 year ago

https://review.dev.storj.io/c/storj/uplink/+/10365 https://review.dev.storj.io/c/storj/uplink/+/10439

shaupt131 commented 1 year ago

I think these ^ should be deployed today, correct? Which are meant to help with diagnosis?

iglesiasbrandon commented 1 year ago

@shaupt131 i think the deployment is happening today