Project

General

Profile

Actions

Bug #14488

open

[keepstore] Do not try to trash blocks that were already trashed in previous sweep

Added by Tom Clegg about 6 years ago. Updated 10 months ago.

Status:
New
Priority:
Normal
Assigned To:
-
Category:
Keep
Target version:
Story points:
-
Release:
Release relationship:
Auto

Description

There is a time interval -- after keep-balance retrieves keepstore's block index, and before keep-balance sends keepstore a new trash list -- during which keepstore is still processing the last trash list it received from a previous keep-balance sweep.

Blocks that get trashed during this interval will typically appear on the new trash list, too. When keepstore runs the new trash list, it logs errors: the blocks can't be trashed because they don't exist.

This produces logs that are hard to distinguish from real errors, and it wastes time and cloud API calls.

Proposed solution:
  • After a block is successfully trashed, add the block ID to a map of recent trashes.
  • When a new trash list is accepted, remove any block ID that appears in the map of recent trashes, and clear the map of recent trashes.
Actions #1

Updated by Peter Amstutz over 3 years ago

  • Target version deleted (To Be Groomed)
Actions #2

Updated by Peter Amstutz almost 2 years ago

  • Release set to 60
Actions #3

Updated by Peter Amstutz 10 months ago

  • Target version set to Future
Actions

Also available in: Atom PDF