Skip to content

Have bulk_update_blobs return a hash->Status map

Jeremiah Bonney requested to merge jbonney/bulk-update-map into master

Before raising this MR, consider whether the following are required, and complete if so:

  • Unit tests
  • Metrics
  • Documentation update(s)

If not required, please explain in brief why not.

Description

This MR updates bulk_update_blobs to return Map[digest.hash, Status] instead of the current ordered List[Status]. This matches what BatchUpdateBlobResponse ends up looking like and simplifies some of the storage implementations which had to do extra work to ensure ordering. This brings it in line with bulk_read_blobs, which returns a similar looking Map[digest.hash, blob].

Merge request reports