hudi-agent commented on code in PR #18840:
URL: https://github.com/apache/hudi/pull/18840#discussion_r3299077142


##########
website/releases/download.md:
##########
@@ -5,6 +5,100 @@ keywords: [ hudi, download ]
 toc: true
 ---
 
+## Release 1.2.0
+
+* Source Release : [Apache Hudi 1.2.0 Source 
Release](https://downloads.apache.org/hudi/1.2.0/hudi-1.2.0.src.tgz) 
([asc](https://downloads.apache.org/hudi/1.2.0/hudi-1.2.0.src.tgz.asc), 
[sha512](https://downloads.apache.org/hudi/1.2.0/hudi-1.2.0.src.tgz.sha512))
+
+* Maven Artifacts:

Review Comment:
   🤖 This section could use a blank line between `</summary>` and the first 
bullet (`* **Spark 4.1**`). All the other collapsible sections in this PR 
(Flink, Query Engines, Utilities, Platform Integrations) include that blank 
line, and without it Markdown often fails to render the list properly inside 
the `<details>` block. Could you mirror the formatting used in the Flink 
Bundles block right below?
   
   <sub><i>- AI-generated; verify before applying. React 👍/👎 to flag 
quality.</i></sub>



##########
website/releases/download.md:
##########
@@ -5,6 +5,100 @@ keywords: [ hudi, download ]
 toc: true
 ---
 
+## Release 1.2.0

Review Comment:
   🤖 The PR description mentions adding Trino/Presto notebook coverage 
alongside existing notebook content, but the diff only updates the Download 
page bundle links. Was the notebook update intended for a separate PR, or 
should this PR be updated to include those changes (or the PR description 
trimmed to match)?
   
   <sub><i>- AI-generated; verify before applying. React 👍/👎 to flag 
quality.</i></sub>



##########
website/releases/download.md:
##########
@@ -5,6 +5,100 @@ keywords: [ hudi, download ]
 toc: true
 ---
 
+## Release 1.2.0
+
+* Source Release : [Apache Hudi 1.2.0 Source 
Release](https://downloads.apache.org/hudi/1.2.0/hudi-1.2.0.src.tgz) 
([asc](https://downloads.apache.org/hudi/1.2.0/hudi-1.2.0.src.tgz.asc), 
[sha512](https://downloads.apache.org/hudi/1.2.0/hudi-1.2.0.src.tgz.sha512))
+
+* Maven Artifacts:
+
+  <details>
+  <summary><strong>Spark Bundles</strong></summary>
+  * **Spark 4.1**
+    * 
[hudi-spark4.1-bundle_2.13](https://repo1.maven.org/maven2/org/apache/hudi/hudi-spark4.1-bundle_2.13/1.2.0/hudi-spark4.1-bundle_2.13-1.2.0.jar)
+
+  * **Spark 4.0**

Review Comment:
   🤖 Could you double-check the complete set of 1.2.0 bundle coordinates 
against what was actually published to Maven Central? In particular: (1) is 
`hudi-spark3.3-bundle_2.12` still being published for 1.2.0, given Spark 3.3 
support has been narrowing in recent releases, and (2) are the Scala 2.13 
variants for `hudi-utilities-bundle` / `hudi-utilities-slim-bundle` / 
`hudi-cli-bundle` all present on Central for 1.2.0? It would be unfortunate to 
publish a dead link for any of these. cc @yihua for visibility on the 1.2.0 
artifact matrix.
   
   <sub><i>- AI-generated; verify before applying. React 👍/👎 to flag 
quality.</i></sub>



##########
website/releases/download.md:
##########
@@ -5,6 +5,100 @@ keywords: [ hudi, download ]
 toc: true
 ---
 
+## Release 1.2.0
+
+* Source Release : [Apache Hudi 1.2.0 Source 
Release](https://downloads.apache.org/hudi/1.2.0/hudi-1.2.0.src.tgz) 
([asc](https://downloads.apache.org/hudi/1.2.0/hudi-1.2.0.src.tgz.asc), 
[sha512](https://downloads.apache.org/hudi/1.2.0/hudi-1.2.0.src.tgz.sha512))
+
+* Maven Artifacts:
+
+  <details>
+  <summary><strong>Spark Bundles</strong></summary>
+  * **Spark 4.1**
+    * 
[hudi-spark4.1-bundle_2.13](https://repo1.maven.org/maven2/org/apache/hudi/hudi-spark4.1-bundle_2.13/1.2.0/hudi-spark4.1-bundle_2.13-1.2.0.jar)
+
+  * **Spark 4.0**
+    * 
[hudi-spark4.0-bundle_2.13](https://repo1.maven.org/maven2/org/apache/hudi/hudi-spark4.0-bundle_2.13/1.2.0/hudi-spark4.0-bundle_2.13-1.2.0.jar)
+
+  * **Spark 3.5**
+    * 
[hudi-spark3.5-bundle_2.13](https://repo1.maven.org/maven2/org/apache/hudi/hudi-spark3.5-bundle_2.13/1.2.0/hudi-spark3.5-bundle_2.13-1.2.0.jar)
+    * 
[hudi-spark3.5-bundle_2.12](https://repo1.maven.org/maven2/org/apache/hudi/hudi-spark3.5-bundle_2.12/1.2.0/hudi-spark3.5-bundle_2.12-1.2.0.jar)
+
+  * **Spark 3.4**
+    * 
[hudi-spark3.4-bundle_2.12](https://repo1.maven.org/maven2/org/apache/hudi/hudi-spark3.4-bundle_2.12/1.2.0/hudi-spark3.4-bundle_2.12-1.2.0.jar)
+
+  * **Spark 3.3**
+    * 
[hudi-spark3.3-bundle_2.12](https://repo1.maven.org/maven2/org/apache/hudi/hudi-spark3.3-bundle_2.12/1.2.0/hudi-spark3.3-bundle_2.12-1.2.0.jar)
+
+  </details>
+
+  <details>
+  <summary><strong>Flink Bundles</strong></summary>
+
+  * **Flink 2.x**
+    * 
[hudi-flink2.1-bundle](https://repo1.maven.org/maven2/org/apache/hudi/hudi-flink2.1-bundle/1.2.0/hudi-flink2.1-bundle-1.2.0.jar)
+    * 
[hudi-flink2.0-bundle](https://repo1.maven.org/maven2/org/apache/hudi/hudi-flink2.0-bundle/1.2.0/hudi-flink2.0-bundle-1.2.0.jar)
+
+  * **Flink 1.x**
+    * 
[hudi-flink1.20-bundle](https://repo1.maven.org/maven2/org/apache/hudi/hudi-flink1.20-bundle/1.2.0/hudi-flink1.20-bundle-1.2.0.jar)
+    * 
[hudi-flink1.19-bundle](https://repo1.maven.org/maven2/org/apache/hudi/hudi-flink1.19-bundle/1.2.0/hudi-flink1.19-bundle-1.2.0.jar)
+    * 
[hudi-flink1.18-bundle](https://repo1.maven.org/maven2/org/apache/hudi/hudi-flink1.18-bundle/1.2.0/hudi-flink1.18-bundle-1.2.0.jar)
+    * 
[hudi-flink1.17-bundle](https://repo1.maven.org/maven2/org/apache/hudi/hudi-flink1.17-bundle/1.2.0/hudi-flink1.17-bundle-1.2.0.jar)
+
+  </details>
+
+  <details>
+  <summary><strong>Query Engines</strong></summary>
+
+  * **Presto**
+    * 
[hudi-presto-bundle](https://repo1.maven.org/maven2/org/apache/hudi/hudi-presto-bundle/1.2.0/hudi-presto-bundle-1.2.0.jar)
+
+  * **Trino**
+    * 
[hudi-trino-bundle](https://repo1.maven.org/maven2/org/apache/hudi/hudi-trino-bundle/1.2.0/hudi-trino-bundle-1.2.0.jar)
+
+  * **Hive**
+    * 
[hudi-hadoop-mr-bundle](https://repo1.maven.org/maven2/org/apache/hudi/hudi-hadoop-mr-bundle/1.2.0/hudi-hadoop-mr-bundle-1.2.0.jar)
+
+  </details>
+
+  <details>
+  <summary><strong>Utilities & Tools</strong></summary>
+
+  * **Hudi Utilities**
+    * 
[hudi-utilities-bundle_2.13](https://repo1.maven.org/maven2/org/apache/hudi/hudi-utilities-bundle_2.13/1.2.0/hudi-utilities-bundle_2.13-1.2.0.jar)
+    * 
[hudi-utilities-bundle_2.12](https://repo1.maven.org/maven2/org/apache/hudi/hudi-utilities-bundle_2.12/1.2.0/hudi-utilities-bundle_2.12-1.2.0.jar)
+    * 
[hudi-utilities-slim-bundle_2.13](https://repo1.maven.org/maven2/org/apache/hudi/hudi-utilities-slim-bundle_2.13/1.2.0/hudi-utilities-slim-bundle_2.13-1.2.0.jar)
+    * 
[hudi-utilities-slim-bundle_2.12](https://repo1.maven.org/maven2/org/apache/hudi/hudi-utilities-slim-bundle_2.12/1.2.0/hudi-utilities-slim-bundle_2.12-1.2.0.jar)
+
+  * **Hudi CLI**
+    * 
[hudi-cli-bundle_2.13](https://repo1.maven.org/maven2/org/apache/hudi/hudi-cli-bundle_2.13/1.2.0/hudi-cli-bundle_2.13-1.2.0.jar)
+    * 
[hudi-cli-bundle_2.12](https://repo1.maven.org/maven2/org/apache/hudi/hudi-cli-bundle_2.12/1.2.0/hudi-cli-bundle_2.12-1.2.0.jar)
+
+  </details>
+
+  <details>
+  <summary><strong>Platform Integrations</strong></summary>
+
+  * **AWS**
+    * 
[hudi-aws-bundle](https://repo1.maven.org/maven2/org/apache/hudi/hudi-aws-bundle/1.2.0/hudi-aws-bundle-1.2.0.jar)
+
+  * **Google Cloud**
+    * 
[hudi-gcp-bundle](https://repo1.maven.org/maven2/org/apache/hudi/hudi-gcp-bundle/1.2.0/hudi-gcp-bundle-1.2.0.jar)
+
+  * **Data Catalogs**
+    * 
[hudi-hive-sync-bundle](https://repo1.maven.org/maven2/org/apache/hudi/hudi-hive-sync-bundle/1.2.0/hudi-hive-sync-bundle-1.2.0.jar)
+    * 
[hudi-datahub-sync-bundle](https://repo1.maven.org/maven2/org/apache/hudi/hudi-datahub-sync-bundle/1.2.0/hudi-datahub-sync-bundle-1.2.0.jar)
+
+  * **Kafka Connect**
+    * 
[hudi-kafka-connect-bundle](https://repo1.maven.org/maven2/org/apache/hudi/hudi-kafka-connect-bundle/1.2.0/hudi-kafka-connect-bundle-1.2.0.jar)
+
+  * **Timeline Server**
+    * 
[hudi-timeline-server-bundle](https://repo1.maven.org/maven2/org/apache/hudi/hudi-timeline-server-bundle/1.2.0/hudi-timeline-server-bundle-1.2.0.jar)
+
+  * **Metaserver**
+    * 
[hudi-metaserver-server-bundle](https://repo1.maven.org/maven2/org/apache/hudi/hudi-metaserver-server-bundle/1.2.0/hudi-metaserver-server-bundle-1.2.0.jar)

Review Comment:
   🤖 Is `hudi-metaserver-server-bundle` the correct artifact id (note the 
doubled `server-server`)? It's worth verifying this coordinate resolves on 
Maven Central for 1.2.0 — if the actual published artifact is named differently 
(e.g. `hudi-metaserver-bundle`), this link will 404.
   
   <sub><i>- AI-generated; verify before applying. React 👍/👎 to flag 
quality.</i></sub>



-- 
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