FUSE: faster rename implementation
This commit is contained in:
@@ -3,11 +3,12 @@ package filesys
|
|||||||
import (
|
import (
|
||||||
"context"
|
"context"
|
||||||
|
|
||||||
|
"github.com/seaweedfs/fuse"
|
||||||
|
"github.com/seaweedfs/fuse/fs"
|
||||||
|
|
||||||
"github.com/chrislusf/seaweedfs/weed/glog"
|
"github.com/chrislusf/seaweedfs/weed/glog"
|
||||||
"github.com/chrislusf/seaweedfs/weed/pb/filer_pb"
|
"github.com/chrislusf/seaweedfs/weed/pb/filer_pb"
|
||||||
"github.com/chrislusf/seaweedfs/weed/util"
|
"github.com/chrislusf/seaweedfs/weed/util"
|
||||||
"github.com/seaweedfs/fuse"
|
|
||||||
"github.com/seaweedfs/fuse/fs"
|
|
||||||
)
|
)
|
||||||
|
|
||||||
func (dir *Dir) Rename(ctx context.Context, req *fuse.RenameRequest, newDirectory fs.Node) error {
|
func (dir *Dir) Rename(ctx context.Context, req *fuse.RenameRequest, newDirectory fs.Node) error {
|
||||||
@@ -19,7 +20,15 @@ func (dir *Dir) Rename(ctx context.Context, req *fuse.RenameRequest, newDirector
|
|||||||
|
|
||||||
glog.V(4).Infof("dir Rename %s => %s", oldPath, newPath)
|
glog.V(4).Infof("dir Rename %s => %s", oldPath, newPath)
|
||||||
|
|
||||||
err := dir.wfs.WithFilerClient(func(client filer_pb.SeaweedFilerClient) error {
|
// find local old entry
|
||||||
|
oldEntry, err := dir.wfs.metaCache.FindEntry(context.Background(), oldPath)
|
||||||
|
if err != nil {
|
||||||
|
glog.V(0).Infof("dir Rename can not find %s => %s : %v", oldPath, err)
|
||||||
|
return fuse.ENOENT
|
||||||
|
}
|
||||||
|
|
||||||
|
// update remote filer
|
||||||
|
err = dir.wfs.WithFilerClient(func(client filer_pb.SeaweedFilerClient) error {
|
||||||
|
|
||||||
request := &filer_pb.AtomicRenameEntryRequest{
|
request := &filer_pb.AtomicRenameEntryRequest{
|
||||||
OldDirectory: dir.FullPath(),
|
OldDirectory: dir.FullPath(),
|
||||||
@@ -30,21 +39,31 @@ func (dir *Dir) Rename(ctx context.Context, req *fuse.RenameRequest, newDirector
|
|||||||
|
|
||||||
_, err := client.AtomicRenameEntry(context.Background(), request)
|
_, err := client.AtomicRenameEntry(context.Background(), request)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
glog.V(0).Infof("dir Rename %s => %s : %v", oldPath, newPath, err)
|
|
||||||
return fuse.EIO
|
return fuse.EIO
|
||||||
}
|
}
|
||||||
|
|
||||||
return nil
|
return nil
|
||||||
|
|
||||||
})
|
})
|
||||||
|
if err != nil {
|
||||||
if err == nil {
|
glog.V(0).Infof("dir Rename %s => %s : %v", oldPath, newPath, err)
|
||||||
|
return fuse.EIO
|
||||||
// fmt.Printf("rename path: %v => %v\n", oldPath, newPath)
|
|
||||||
dir.wfs.fsNodeCache.Move(oldPath, newPath)
|
|
||||||
delete(dir.wfs.handles, oldPath.AsInode())
|
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// TODO: replicate renaming logic on filer
|
||||||
|
if err := dir.wfs.metaCache.DeleteEntry(context.Background(), oldPath); err != nil {
|
||||||
|
glog.V(0).Infof("dir Rename delete local %s => %s : %v", oldPath, newPath, err)
|
||||||
|
return fuse.EIO
|
||||||
|
}
|
||||||
|
oldEntry.FullPath = newPath
|
||||||
|
if err := dir.wfs.metaCache.InsertEntry(context.Background(), oldEntry); err != nil {
|
||||||
|
glog.V(0).Infof("dir Rename insert local %s => %s : %v", oldPath, newPath, err)
|
||||||
|
return fuse.EIO
|
||||||
|
}
|
||||||
|
|
||||||
|
// fmt.Printf("rename path: %v => %v\n", oldPath, newPath)
|
||||||
|
dir.wfs.fsNodeCache.Move(oldPath, newPath)
|
||||||
|
delete(dir.wfs.handles, oldPath.AsInode())
|
||||||
|
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user