@aganders3 Indirectly related to this PR. I think there might be a bug in the chunk manager:
getOrCreateStore doesn't seem to manage failure. I think that if source.open() rejects, pendingStores_ keeps a rejected promise forever so future calls keep failing and we never retry. We should clear pendingStores_ in a finally and log the error.
Originally posted by @shlomnissan in #553 (comment)
@aganders3 Indirectly related to this PR. I think there might be a bug in the chunk manager:
getOrCreateStoredoesn't seem to manage failure. I think that ifsource.open()rejects,pendingStores_keeps a rejected promise forever so future calls keep failing and we never retry. We should clearpendingStores_in a finally and log the error.Originally posted by @shlomnissan in #553 (comment)