🎨 Skip persisting referencing docs when block ref anchor text is unchanged #18058

Author: TCOTC

kernel/av/av.go

@@ -651,14 +651,19 @@ func SaveAttributeView(av *AttributeView) (err error) {
 		return
 	}
 
-	// 缓存命中且字节相等时跳过落盘，避免无变更的重复写入
+	// 缓存与待写入数据一致时跳过落盘；缓存未命中时再读盘比对，避免无变更的重复写入
+	avJSONPath := GetAttributeViewDataPath(av.ID)
 	if cachedData, ok := cache.GetAVData(av.ID); ok {
 		if len(cachedData) == len(data) && bytes.Equal(cachedData, data) {
 			return
 		}
+	} else {
+		if diskData, readErr := filelock.ReadFile(avJSONPath); nil == readErr && len(diskData) == len(data) && bytes.Equal(diskData, data) {
+			cache.SetAVData(av.ID, data)
+			return
+		}
 	}
 
-	avJSONPath := GetAttributeViewDataPath(av.ID)
 	if err = util.WriteFileByMmap(avJSONPath, data); nil != err {
 		if err = filelock.WriteFile(avJSONPath, data); nil != err {
 			logging.LogErrorf("save attribute view [%s] failed: %s", av.ID, err)

kernel/filesys/tree.go

@@ -246,11 +246,16 @@ func WriteTree(tree *parse.Tree) (size uint64, err error) {
 		return
 	}
 
-	cachedData, ok := cache.GetTreeData(tree.ID)
-	if ok {
+	// 缓存与待写入数据一致时跳过落盘；缓存未命中时再读盘比对，避免无变更的重复写入
+	if cachedData, ok := cache.GetTreeData(tree.ID); ok {
 		if len(cachedData) == len(data) && bytes.Equal(cachedData, data) {
 			return
 		}
+	} else {
+		if diskData, readErr := filelock.ReadFile(filePath); nil == readErr && len(diskData) == len(data) && bytes.Equal(diskData, data) {
+			cache.SetTreeData(tree.ID, data)
+			return
+		}
 	}
 
 	if err = util.WriteFileByMmap(filePath, data); nil != err {

kernel/model/blockial.go

@@ -221,13 +221,26 @@ func setNodeAttrs(node *ast.Node, tree *parse.Tree, nameValues map[string]string
 		ReloadFiletree()
 	}
 
-	go func() {
-		sql.FlushQueue()
-		refreshDynamicRefText(node, tree)
-	}()
+	if attrsAffectRefText(nameValues) {
+		go func() {
+			sql.FlushQueue()
+			refreshDynamicRefText(node, tree)
+		}()
+	}
 	return
 }
 
+// attrsAffectRefText 判断本次属性变更是否可能影响引用处的动态锚文本。
+func attrsAffectRefText(nameValues map[string]string) bool {
+	for name := range nameValues {
+		switch strings.ToLower(name) {
+		case "name", "title":
+			return true
+		}
+	}
+	return false
+}
+
 func setNodeAttrsWithTx(tx *Transaction, node *ast.Node, tree *parse.Tree, nameValues map[string]string) (err error) {
 	oldAttrs, err := setNodeAttrs0(node, nameValues)
 	if err != nil {

kernel/model/transaction.go

@@ -2119,21 +2119,19 @@ func updateRefText(refNode *ast.Node, changedDefNodes map[string]*ast.Node) (cha
 				return ast.WalkSkipChildren
 			}
 
-			changed = true
 			if "d" == subtype {
-				refText = strings.TrimSpace(getNodeRefText(defNode))
-				if "" == refText {
-					refText = n.TextMarkBlockRefID
+				newRefText := strings.TrimSpace(getNodeRefText(defNode))
+				if "" == newRefText {
+					newRefText = n.TextMarkBlockRefID
 				}
-				treenode.SetDynamicBlockRefText(n, refText)
+				if strings.TrimSpace(refText) == newRefText {
+					return ast.WalkContinue
+				}
+				treenode.SetDynamicBlockRefText(n, newRefText)
+				changed = true
+				refText = newRefText
+				defNodes = append(defNodes, &changedDefNode{id: defID, refText: refText, refType: "ref-" + subtype})
 			}
-			defNodes = append(defNodes, &changedDefNode{id: defID, refText: refText, refType: "ref-" + subtype})
-			return ast.WalkContinue
-		} else if treenode.IsEmbedBlockRef(n) {
-			defID := treenode.GetEmbedBlockRef(n)
-			changed = true
-			defNodes = append(defNodes, &changedDefNode{id: defID, refType: "embed"})
-			return ast.WalkContinue
 		}
 		return ast.WalkContinue
 	})