Copilot commented on code in PR #12641: URL: https://github.com/apache/gravitino/pull/12641#discussion_r4069477017
########## docs/flink-connector/flink-connector.md: ########## @@ -14,17 +14,23 @@ The connector is published as a version-specific runtime JAR for each supported ## Capabilities -1. Supports [Hive catalog](flink-catalog-hive.md) +1. Supports [Hive catalog](flink-catalog-hive.md) (Flink 1.18, 1.19, and 1.20 only; not yet available on + Flink 2.x — see [Prerequisites](#prerequisites)) 2. Supports [Iceberg catalog](flink-catalog-iceberg.md) 3. Supports [Paimon catalog](flink-catalog-paimon.md) 4. Supports [Jdbc catalog](flink-catalog-jdbc.md) 5. Supports most DDL and DML SQLs. ## Prerequisites -* Scala 2.12 -* Flink 1.18, 1.19, or 1.20 -* JDK 8, 11 or 17 +* Scala 2.12 (Flink 1.18, 1.19, and 1.20 only; Flink 2.x removed the Scala APIs) +* Flink 1.18, 1.19, 1.20, or 2.1 +* JDK 8, 11 or 17 for Flink 1.x; JDK 17 for Flink 2.x Review Comment: The overview now advertises Flink 2.1 support for Iceberg, Paimon, and JDBC, but the linked catalog guides still contain only 1.18–1.20 dependency tables. Users therefore have no documented 2.1 client artifacts to pair with this runtime, especially the split `flink-connector-jdbc-core`/`-mysql`/`-postgres` artifacts introduced here. Add 2.1 rows to those guides before publishing this support claim. -- 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]
