Copilot commented on code in PR #9467:
URL: https://github.com/apache/seatunnel/pull/9467#discussion_r2155952624
##########
seatunnel-e2e/seatunnel-connector-v2-e2e/connector-paimon-e2e/src/test/java/org/apache/seatunnel/e2e/connector/paimon/PaimonIT.java:
##########
@@ -34,9 +35,21 @@
import java.nio.file.Path;
@DisabledOnContainer(
- value = TestContainerId.FLINK_1_13,
- disabledReason = "Paimon does not support flink 1.13")
-public class PaimonIT extends TestSuiteBase {
+ value = {},
+ type = {EngineType.SPARK, EngineType.FLINK},
+ disabledReason =
+ "Spark and Flink engine can not auto create paimon table on
worker node in local file(e.g flink tm) by savemode feature which can lead
error")
Review Comment:
Consider rephrasing the disabledReason to clearly explain the limitation
with auto-creating paimon tables on local worker nodes (e.g., detailing the
save mode issue) for better clarity to future maintainers.
```suggestion
"Spark and Flink engines cannot automatically create Paimon
tables on local worker nodes due to limitations with the save mode feature. The
save mode requires the table to exist beforehand, which can cause errors when
running jobs on local file systems (e.g., Flink TaskManager).")
```
--
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]
