use a short-lived lock

weed/cluster/lock_client.go

@@ -40,9 +40,19 @@ type LiveLock struct {
 	lc             *LockClient
 }
 
-// NewLock creates a lock with a very long duration
+// NewShortLivedLock creates a lock with a 5-second duration
-func (lc *LockClient) NewLock(key string, owner string) (lock *LiveLock) {
+func (lc *LockClient) NewShortLivedLock(key string, owner string) (lock *LiveLock) {
-	return lc.doNewLock(key, lock_manager.MaxDuration, owner)
+	lock = &LiveLock{
+		key:            key,
+		filer:          lc.seedFiler,
+		cancelCh:       make(chan struct{}),
+		expireAtNs:     time.Now().Add(5*time.Second).UnixNano(),
+		grpcDialOption: lc.grpcDialOption,
+		owner:          owner,
+		lc:             lc,
+	}
+	lock.retryUntilLocked(5*time.Second)
+	return
 }
 
 // StartLock starts a goroutine to lock the key and returns immediately.
@@ -57,40 +67,15 @@ func (lc *LockClient) StartLock(key string, owner string) (lock *LiveLock) {
 		lc:             lc,
 	}
 	go func() {
-		lock.CreateLock(lock_manager.MaxDuration)
+		lock.retryUntilLocked(lock_manager.MaxDuration)
 		lc.keepLock(lock)
 	}()
 	return
 }
 
-func (lc *LockClient) doNewLock(key string, lockDuration time.Duration, owner string) (lock *LiveLock) {
+func (lock *LiveLock) retryUntilLocked(lockDuration time.Duration) {
-	lock = &LiveLock{
-		key:            key,
-		filer:          lc.seedFiler,
-		cancelCh:       make(chan struct{}),
-		expireAtNs:     time.Now().Add(lockDuration).UnixNano(),
-		grpcDialOption: lc.grpcDialOption,
-		owner:          owner,
-		lc:             lc,
-	}
-	var needRenewal bool
-	if lockDuration > lc.maxLockDuration {
-		lockDuration = lc.maxLockDuration
-		needRenewal = true
-	}
-
-	lock.CreateLock(lockDuration)
-
-	if needRenewal {
-		go lc.keepLock(lock)
-	}
-
-	return
-}
-
-func (lock *LiveLock) CreateLock(lockDuration time.Duration) {
 	util.RetryUntil("create lock:"+lock.key, func() error {
-		return lock.DoLock(lockDuration)
+		return lock.AttemptToLock(lockDuration)
 	}, func(err error) (shouldContinue bool) {
 		if err != nil {
 			glog.Warningf("create lock %s: %s", lock.key, err)
@@ -99,7 +84,7 @@ func (lock *LiveLock) CreateLock(lockDuration time.Duration) {
 	})
 }
 
-func (lock *LiveLock) DoLock(lockDuration time.Duration) error {
+func (lock *LiveLock) AttemptToLock(lockDuration time.Duration) error {
 	errorMessage, err := lock.doLock(lockDuration)
 	if err != nil {
 		time.Sleep(time.Second)
@@ -117,11 +102,13 @@ func (lock *LiveLock) IsLocked() bool {
 	return lock!=nil && lock.isLocked
 }
 
-func (lock *LiveLock) StopLock() error {
+func (lock *LiveLock) StopShortLivedLock() error {
-	close(lock.cancelCh)
 	if !lock.isLocked {
 		return nil
 	}
+	defer func() {
+		lock.isLocked = false
+	}()
 	return pb.WithFilerClient(false, 0, lock.filer, lock.grpcDialOption, func(client filer_pb.SeaweedFilerClient) error {
 		_, err := client.DistributedUnlock(context.Background(), &filer_pb.UnlockRequest{
 			Name:       lock.key,

weed/mq/broker/broker_server.go

@@ -92,8 +92,8 @@ func NewMessageBroker(option *MessageQueueBrokerOption, grpcDialOption grpc.Dial
 				glog.V(0).Infof("BrokerConnectToBalancer: %v", err)
 			}
 			time.Sleep(time.Second)
-			if err := mqBroker.lockAsBalancer.DoLock(lock_manager.MaxDuration); err != nil {
+			if err := mqBroker.lockAsBalancer.AttemptToLock(lock_manager.MaxDuration); err != nil {
-				glog.V(0).Infof("DoLock: %v", err)
+				glog.V(0).Infof("AttemptToLock: %v", err)
 			}
 		}
 	}()

weed/server/filer_grpc_server.go

@@ -245,8 +245,8 @@ func (fs *FilerServer) AppendToEntry(ctx context.Context, req *filer_pb.AppendTo
 	fullpath := util.NewFullPath(req.Directory, req.EntryName)
 
 	lockClient := cluster.NewLockClient(fs.grpcDialOption, fs.option.Host)
-	lock := lockClient.NewLock(string(fullpath), string(fs.option.Host))
+	lock := lockClient.NewShortLivedLock(string(fullpath), string(fs.option.Host))
-	defer lock.StopLock()
+	defer lock.StopShortLivedLock()
 
 	var offset int64 = 0
 	entry, err := fs.filer.FindEntry(ctx, fullpath)