Fix an issue where negative lock sequence numbers are incorrectly mad…

…e positive.
openconfig · Sep 1, 2021 · 2f4106e · 2f4106e
1 parent d0ed5a2
commit 2f4106e
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/gateway/locking/zookeeper.go b/gateway/locking/zookeeper.go
@@ -102,7 +102,7 @@ func (l *ZookeeperNonBlockingLock) ID() string {
 }
 
 func parseSeq(path string) (int, error) {
-	parts := strings.Split(path, "-")
+	parts := strings.Split(path, ":")
 	return strconv.Atoi(parts[len(parts)-1])
 }
 
@@ -127,7 +127,7 @@ func (l *ZookeeperNonBlockingLock) try() (bool, error) {
 		return true, zk.ErrDeadlock
 	}
 
-	prefix := fmt.Sprintf("%s/lock-", l.id)
+	prefix := fmt.Sprintf("%s/lock:", l.id)
 
 	// Attempt to add a sequence to the tree
 	path, err := l.conn.CreateProtectedEphemeralSequential(prefix, []byte(l.member), l.acl)