Simplify bucket delete - remove only bucket directory, no need to recursively traverse

This commit is contained in:
Harshavardhana 2015-10-30 16:03:18 -07:00
parent 7d38967f22
commit baf66988e9
2 changed files with 0 additions and 35 deletions

View File

@ -41,12 +41,6 @@ func (fs Filesystem) DeleteBucket(bucket string) *probe.Error {
if _, err := os.Stat(bucketDir); os.IsNotExist(err) {
return probe.NewError(BucketNotFound{Bucket: bucket})
}
if err := RemoveAllDirs(bucketDir); err != nil {
if err == ErrDirNotEmpty || strings.Contains(err.Error(), "directory not empty") {
return probe.NewError(BucketNotEmpty{Bucket: bucket})
}
return probe.NewError(err)
}
if err := os.Remove(bucketDir); err != nil {
if strings.Contains(err.Error(), "directory not empty") {
return probe.NewError(BucketNotEmpty{Bucket: bucket})

View File

@ -43,35 +43,6 @@ func isDirEmpty(dirname string) (bool, *probe.Error) {
return true, nil
}
// RemoveAllDirs - removes only itself and all subdirectories
func RemoveAllDirs(path string) error {
allFiles := func(fp string, fl os.FileInfo, err error) error {
if fp == path {
return nil
}
if fl.Mode().IsRegular() || fl.Mode()&os.ModeSymlink == os.ModeSymlink {
return ErrDirNotEmpty
}
if fl.Mode().IsDir() {
if err := os.Remove(fp); err != nil {
if os.IsNotExist(err) {
return nil
}
return err
}
}
return nil
}
err := WalkUnsorted(path, allFiles)
if err != nil {
if os.IsNotExist(err) {
return nil
}
return err
}
return nil
}
// Walk walks the file tree rooted at root, calling walkFn for each file or
// directory in the tree, including root.
func Walk(root string, walkFn WalkFunc) error {