dolphinscheduler icon indicating copy to clipboard operation
dolphinscheduler copied to clipboard

[Feature-16992][dolphinscheduler-task-seatunnel] Implement added seatunnel configuration generation in the form mode

Open JeremyXin opened this issue 9 months ago • 7 comments
trafficstars

Purpose of the pull request

Implement added seatunnel configuration generation in the form mode. Try to implement issues #16992. Full ideas and front and backend effection picture can be found in the pdf file below. DolphinScheduler Seatunnel plugin form configuration scheme.pdf

If you have any questions, please contact me in time. Thank you for your review!

close #16992

Brief change log

Modify the front and backend code of the dolphinscheduler-task-seatunnel plugin

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows: SeatunnelTaskTest.testSeatunnelConfigGeneration() method.

(or)

Pull Request Notice

Pull Request Notice

If your pull request contains incompatible change, you should also add it to docs/docs/en/guide/upgrade/incompatible.md

JeremyXin avatar Feb 02 '25 08:02 JeremyXin

Thanks for opening this pull request! Please check out our contributing guidelines. (https://github.com/apache/dolphinscheduler/blob/dev/docs/docs/en/contribute/join/pull-request.md)

boring-cyborg[bot] avatar Feb 02 '25 08:02 boring-cyborg[bot]

Hi @JeremyXin , thanks for your contribution. Can you copy and paste the contents of the pdf into the description with english, which will make it easier for others to participate in the discussion and review?

SbloodyS avatar Feb 02 '25 13:02 SbloodyS

Hi @JeremyXin , thanks for your contribution. Can you copy and paste the contents of the pdf into the description with english, which will make it easier for others to participate in the discussion and review?

Ok,I have updated the English version of the document and optimized the code. Thank you for your reply.

JeremyXin avatar Feb 04 '25 12:02 JeremyXin

How can I retry analysis of this Pull-Request directly on SonarQube Cloud?

JeremyXin avatar Feb 06 '25 04:02 JeremyXin

How can I retry analysis of this Pull-Request directly on SonarQube Cloud?

to First-time contributor, it need to trigger CI by committer manually

davidzollo avatar Feb 11 '25 01:02 davidzollo

I have added unit test code to improve coverage on new code and may need to re-execute SonarQube Cloud. Thanks!

JeremyXin avatar Feb 11 '25 03:02 JeremyXin

Please retry analysis of this Pull-Request directly on SonarQube Cloud

sonarqubecloud[bot] avatar Feb 11 '25 10:02 sonarqubecloud[bot]

This pull request has been automatically marked as stale because it has not had recent activity for 120 days. It will be closed in 7 days if no further activity occurs.

github-actions[bot] avatar Jul 29 '25 00:07 github-actions[bot]

This pull request has been closed because it has not had recent activity. You could reopen it if you try to continue your work, and anyone who are interested in it are encouraged to continue work on this pull request.

github-actions[bot] avatar Aug 06 '25 00:08 github-actions[bot]