Skip to content

Commit

Permalink
fix tier1 not deleting partial stores in some conditions
Browse files Browse the repository at this point in the history
  • Loading branch information
sduchesneau committed Dec 21, 2023
1 parent ddf7386 commit a355c50
Showing 1 changed file with 4 additions and 7 deletions.
11 changes: 4 additions & 7 deletions orchestrator/stage/squash.go
Original file line number Diff line number Diff line change
Expand Up @@ -84,13 +84,10 @@ func (s *Stages) singleSquash(stage *Stage, modState *ModuleState, mergeUnit Uni
modState.lastBlockInStore = rng.ExclusiveEndBlock
metrics.mergeEnd = time.Now()

// Delete partial store
if reqctx.Details(s.ctx).ProductionMode || segmentEndsOnInterval { /* FIXME: compute this elsewhere? */
s.logger.Info("deleting store", zap.Stringer("store", partialKV))
stage.asyncWork.Go(func() error {
return partialKV.DeleteStore(s.ctx, partialFile)
})
}
s.logger.Info("deleting partial store", zap.Stringer("store", partialKV))
stage.asyncWork.Go(func() error {
return partialKV.DeleteStore(s.ctx, partialFile)
})

// Flush full store
if segmentEndsOnInterval {
Expand Down

0 comments on commit a355c50

Please sign in to comment.