wuchong commented on code in PR #2300:
URL: https://github.com/apache/fluss/pull/2300#discussion_r2659707029


##########
website/docs/table-design/merge-engines/default.md:
##########
@@ -9,7 +9,7 @@ sidebar_position: 2
 ## Overview
 
 The **Default Merge Engine** behaves as a LastRow merge engine that retains 
the latest record for a given primary key. It supports all the operations: 
`INSERT`, `UPDATE`, `DELETE`.
-Additionally, the default merge engine supports [Partial 
Update](table-design/table-types/pk-table/index.md#partial-update), which 
preserves the latest values for the specified update columns.
+Additionally, the default merge engine supports [Partial 
Update](../table-types/pk-table/index.md#partial-update), which preserves the 
latest values for the specified update columns.

Review Comment:
   The original link should still work. 



##########
website/docs/table-design/table-types/pk-table/index.md:
##########
@@ -82,10 +82,10 @@ However, users can specify a different merge engine to 
customize the merging beh
 
 The following merge engines are supported:
 
-1. [Default Merge Engine (LastRow)](merge-engines/default.md)
-2. [FirstRow Merge Engine](merge-engines/first-row.md)
-3. [Versioned Merge Engine](merge-engines/versioned.md)
-4. [Aggregation Merge Engine](merge-engines/aggregation.md)
+1. [Default Merge Engine (LastRow)](../../merge-engines/default.md)

Review Comment:
   It's suggested to use paths relative to the `docs/` directory
    `table-design/merge-engines/default.md`, see 
https://fluss.apache.org/community/how-to-contribute/contribute-docs/#documentation-linking



##########
website/docs/table-design/merge-engines/aggregation.md:
##########
@@ -1015,5 +1015,5 @@ For detailed information about Exactly-Once 
implementation, please refer to: [FI
 - [Default Merge Engine](./default.md)
 - [FirstRow Merge Engine](./first-row.md)
 - [Versioned Merge Engine](./versioned.md)
-- [Primary Key Tables](../index.md)
-- [Fluss Client API](../../../../apis/java-client.md)
+- [Primary Key Tables](../table-types/pk-table/index.md)
+- [Fluss Client API](../../apis/java-client.md)

Review Comment:
   ditto. It's suggested to use paths relative to the `docs/` directory.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to