weifuwan commented on PR #11058: URL: https://github.com/apache/seatunnel/pull/11058#issuecomment-4729381188
I reviewed this PR mainly from the first-time user onboarding perspective. Overall, I think the direction is good. Adding a clear "Run your first job" entry and several practical getting-started recipes can make the SeaTunnel documentation much friendlier for new users. I especially like that the recipes cover common real-world scenarios, such as MySQL CDC to Doris, JDBC to S3, Kafka to Iceberg, HTTP to JDBC, File to StarRocks, and multi-table CDC. I also checked the previous review concerns around the HTTP connector plugin name and the FieldMapper expected output. The latest revision appears to have aligned these points with the actual repository behavior: - HTTP source uses `connector-http-base` - FieldMapper output uses the mapped field name `new_name` From my side, I do not see a blocking issue in this docs-only PR. The structure is clear, the onboarding path is useful, and the current changes look good to me. LGTM. -- 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]
