chore: add additional error logging when deleting shard (#24038)
* chore: add additional error logging when deleting shard * chore: better logging messagetest/monitor-ci-415
parent
06a59020d0
commit
8ad6e17265
|
@ -803,7 +803,7 @@ func (s *Store) DeleteShard(shardID uint64) error {
|
||||||
|
|
||||||
ss := index.SeriesIDSet()
|
ss := index.SeriesIDSet()
|
||||||
|
|
||||||
s.walkShards(shards, func(sh *Shard) error {
|
err = s.walkShards(shards, func(sh *Shard) error {
|
||||||
index, err := sh.Index()
|
index, err := sh.Index()
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return err
|
return err
|
||||||
|
@ -813,13 +813,20 @@ func (s *Store) DeleteShard(shardID uint64) error {
|
||||||
return nil
|
return nil
|
||||||
})
|
})
|
||||||
|
|
||||||
|
if err != nil {
|
||||||
|
s.Logger.Error("error walking shards during DeleteShard operation", zap.Error(err))
|
||||||
|
}
|
||||||
|
|
||||||
// Remove any remaining series in the set from the series file, as they don't
|
// Remove any remaining series in the set from the series file, as they don't
|
||||||
// exist in any of the database's remaining shards.
|
// exist in any of the database's remaining shards.
|
||||||
if ss.Cardinality() > 0 {
|
if ss.Cardinality() > 0 {
|
||||||
sfile := s.seriesFile(db)
|
sfile := s.seriesFile(db)
|
||||||
if sfile != nil {
|
if sfile != nil {
|
||||||
ss.ForEach(func(id uint64) {
|
ss.ForEach(func(id uint64) {
|
||||||
sfile.DeleteSeriesID(id)
|
err = sfile.DeleteSeriesID(id)
|
||||||
|
if err != nil {
|
||||||
|
s.Logger.Error("error deleting series id during DeleteShard operation", zap.Uint64("id", id), zap.Error(err))
|
||||||
|
}
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue