[GitHub] [hudi] danny0405 commented on a diff in pull request #8055: [HUDI-4849][DOCS] Remove default value for mandatory record key field

["danny0405 (via GitHub)" <gi...@apache.org>]

danny0405 commented on code in PR #8055:
URL: https://github.com/apache/hudi/pull/8055#discussion_r1118411261


##########
website/docs/basic_configurations.md:
##########
@@ -74,7 +74,7 @@ Options useful for writing tables via `write.format.option(...)`
 > Record key field. Value to be used as the `recordKey` component of `HoodieKey`.
 Actual value will be obtained by invoking .toString() on the field value. Nested fields can be specified using
 the dot notation eg: `a.b.c`<br></br>
-> **Default Value**: uuid (Optional)<br></br>
+> **Default Value**:  N/A (Required)<br></br>
 > `Config Param: RECORDKEY_FIELD`<br></br>

Review Comment:
   The default primary key `uuid` is broken now?



-- 
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: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org