Follow-up from "Add support for partial downloads to the renter package"
The following discussions from !3349 (merged) should be addressed:
-
@lukechampine started a discussion: Let's circle back to this and decide how we want to handle caching post-partial-downloads.
-
@DavidVorick started a discussion: I know this TODO isn't from your MR, but it's an optimization TODO and therefore can be removed.
-
@DavidVorick started a discussion: poor form / bad code smell to give a variable the same name as one of your helper functions, since the variable will shadow the function. It's not a huge deal but might make more sense to call them
btr
orrecoveryBytes
just to ensure we aren't doing any shadowing. -
@DavidVorick started a discussion: (+1 comment) Need a separate issue to figure out new caching strategy, though the strategy might just be to get rid of all caching.