diff options
| author | Chris Lu <chrislusf@users.noreply.github.com> | 2025-10-25 00:09:18 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-10-25 00:09:18 -0700 |
| commit | 6a8c53bc44beb057f64d5ba1f7ac026f8410fe04 (patch) | |
| tree | 2b3ceed66edd6dd141370bad3fbfce07886840bd /weed/shell/command_volume_fsck.go | |
| parent | 37af41fbfeaf2e9830e25b658b8bed409fa6fae6 (diff) | |
| download | seaweedfs-6a8c53bc44beb057f64d5ba1f7ac026f8410fe04.tar.xz seaweedfs-6a8c53bc44beb057f64d5ba1f7ac026f8410fe04.zip | |
Filer: batch deletion operations to return individual error results (#7382)
* batch deletion operations to return individual error results
Modify batch deletion operations to return individual error results instead of one aggregated error, enabling better tracking of which specific files failed to delete (helping reduce orphan file issues).
* Simplified logging logic
* Optimized nested loop
* handles the edge case where the RPC succeeds but connection cleanup fails
* simplify
* simplify
* ignore 'not found' errors here
Diffstat (limited to 'weed/shell/command_volume_fsck.go')
| -rw-r--r-- | weed/shell/command_volume_fsck.go | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/weed/shell/command_volume_fsck.go b/weed/shell/command_volume_fsck.go index a8ccebf84..878109ecb 100644 --- a/weed/shell/command_volume_fsck.go +++ b/weed/shell/command_volume_fsck.go @@ -702,9 +702,8 @@ func (c *commandVolumeFsck) purgeFileIdsForOneVolume(volumeId uint32, fileIds [] go func(server pb.ServerAddress, fidList []string) { defer wg.Done() - if deleteResults, deleteErr := operation.DeleteFileIdsAtOneVolumeServer(server, c.env.option.GrpcDialOption, fidList, false); deleteErr != nil { - err = deleteErr - } else if deleteResults != nil { + deleteResults := operation.DeleteFileIdsAtOneVolumeServer(server, c.env.option.GrpcDialOption, fidList, false) + if deleteResults != nil { resultChan <- deleteResults } |