From 59e0ae11b4955a113c2b14eb6d26cf9789d0d1ea Mon Sep 17 00:00:00 2001
From: HunterXHunter <cnlingmingqiang@gmail.com>
Date: Sun, 28 Jul 2024 11:18:17 +0800
Subject: [PATCH] [doc] Fix duplicate possible values in the configuration doc.
 (#3821)

---
 .../generated/core_configuration.html         |  4 +-
 .../java/org/apache/paimon/CoreOptions.java   | 40 +++----------------
 2 files changed, 8 insertions(+), 36 deletions(-)

diff --git a/docs/layouts/shortcodes/generated/core_configuration.html b/docs/layouts/shortcodes/generated/core_configuration.html
index 0ecdbd634180..d6eb502353f5 100644
--- a/docs/layouts/shortcodes/generated/core_configuration.html
+++ b/docs/layouts/shortcodes/generated/core_configuration.html
@@ -519,7 +519,7 @@
             <td><h5>partition.expiration-strategy</h5></td>
             <td style="word-wrap: break-word;">values-time</td>
             <td><p>Enum</p></td>
-            <td>Specifies the expiration strategy for partition expiration.<br />Possible values:<ul><li>values-time: A partition expiration policy that compares the time extracted from the partition value with the current time.</li></ul><ul><li>update-time: A partition expiration policy that compares the last update time of the partition with the current time.</li></ul><br /><br />Possible values:<ul><li>"values-time": The strategy compares the time extracted from the partition value with the current time.</li><li>"update-time": The strategy compares the last update time of the partition with the current time.</li></ul></td>
+            <td>The strategy determines how to extract the partition time and compare it with the current time.<br /><br />Possible values:<ul><li>"values-time": This strategy compares the time extracted from the partition value with the current time.</li><li>"update-time": This strategy compares the last update time of the partition with the current time.</li></ul></td>
         </tr>
         <tr>
             <td><h5>partition.expiration-time</h5></td>
@@ -748,7 +748,7 @@
             <td><h5>streaming-read-mode</h5></td>
             <td style="word-wrap: break-word;">(none)</td>
             <td><p>Enum</p></td>
-            <td>The mode of streaming read that specifies to read the data of table file or log<br /><br />Possible values:<br /><ul><li>file: Reads from the data of table file store.</li></ul><ul><li>log: Read from the data of table log store.</li></ul><br /><br />Possible values:<ul><li>"log": Reads from the log store.</li><li>"file": Reads from the file store.</li></ul></td>
+            <td>The mode of streaming read that specifies to read the data of table file or log.<br /><br />Possible values:<ul><li>"log": Read from the data of table log store.</li><li>"file": Read from the data of table file store.</li></ul></td>
         </tr>
         <tr>
             <td><h5>streaming-read-overwrite</h5></td>
diff --git a/paimon-common/src/main/java/org/apache/paimon/CoreOptions.java b/paimon-common/src/main/java/org/apache/paimon/CoreOptions.java
index ade36f2959c8..685634e2650a 100644
--- a/paimon-common/src/main/java/org/apache/paimon/CoreOptions.java
+++ b/paimon-common/src/main/java/org/apache/paimon/CoreOptions.java
@@ -707,20 +707,7 @@ public class CoreOptions implements Serializable {
                     .enumType(PartitionExpireStrategy.class)
                     .defaultValue(PartitionExpireStrategy.VALUES_TIME)
                     .withDescription(
-                            Description.builder()
-                                    .text(
-                                            "Specifies the expiration strategy for partition expiration.")
-                                    .linebreak()
-                                    .text("Possible values:")
-                                    .list(
-                                            text(
-                                                    PartitionExpireStrategy.VALUES_TIME.value
-                                                            + ": A partition expiration policy that compares the time extracted from the partition value with the current time."))
-                                    .list(
-                                            text(
-                                                    PartitionExpireStrategy.UPDATE_TIME.value
-                                                            + ": A partition expiration policy that compares the last update time of the partition with the current time."))
-                                    .build());
+                            "The strategy determines how to extract the partition time and compare it with the current time.");
 
     public static final ConfigOption<Duration> PARTITION_EXPIRATION_TIME =
             key("partition.expiration-time")
@@ -898,22 +885,7 @@ public class CoreOptions implements Serializable {
                     .enumType(StreamingReadMode.class)
                     .noDefaultValue()
                     .withDescription(
-                            Description.builder()
-                                    .text(
-                                            "The mode of streaming read that specifies to read the data of table file or log")
-                                    .linebreak()
-                                    .linebreak()
-                                    .text("Possible values:")
-                                    .linebreak()
-                                    .list(
-                                            text(
-                                                    StreamingReadMode.FILE.getValue()
-                                                            + ": Reads from the data of table file store."))
-                                    .list(
-                                            text(
-                                                    StreamingReadMode.LOG.getValue()
-                                                            + ": Read from the data of table log store."))
-                                    .build());
+                            "The mode of streaming read that specifies to read the data of table file or log.");
 
     public static final ConfigOption<Duration> CONSUMER_EXPIRATION_TIME =
             key("consumer.expiration-time")
@@ -2249,8 +2221,8 @@ public InlineElement getDescription() {
 
     /** Specifies the type for streaming read. */
     public enum StreamingReadMode implements DescribedEnum {
-        LOG("log", "Reads from the log store."),
-        FILE("file", "Reads from the file store.");
+        LOG("log", "Read from the data of table log store."),
+        FILE("file", "Read from the data of table file store.");
 
         private final String value;
         private final String description;
@@ -2601,11 +2573,11 @@ public InlineElement getDescription() {
     public enum PartitionExpireStrategy implements DescribedEnum {
         VALUES_TIME(
                 "values-time",
-                "The strategy compares the time extracted from the partition value with the current time."),
+                "This strategy compares the time extracted from the partition value with the current time."),
 
         UPDATE_TIME(
                 "update-time",
-                "The strategy compares the last update time of the partition with the current time.");
+                "This strategy compares the last update time of the partition with the current time.");
 
         private final String value;