[DSIP-27][Task Plugin] Some improvements of JAVA task plugin - Githubissues

apache / dolphinscheduler

Apache DolphinScheduler is the modern data orchestration platform. Agile to create high performance workflow with low-code

https://dolphinscheduler.apache.org/

Apache License 2.0

12.73k stars 4.58k forks source link

[DSIP-27][Task Plugin] Some improvements of JAVA task plugin #16542

Open ailiujiarui opened 1 month ago

ailiujiarui commented 1 month ago

Purpose of the pull request

Update the Java task plugin close:https://github.com/apache/dolphinscheduler/issues/15819

Brief change log

Deprecate write java code in JAVA task
Rename the JAR to FATJAR Only the displayed name has been modified, the functionality remains unchanged.
Add the new type NORMALJAR Allow users to upload normal type of jar files, which require external libraries to run properly. Users do not need to package all libraries and runtime files into a fat jar each time, making the submission of jar files more flexible and convenient.
Update the tests Delete tests about JAVA type and add the test of NORMAL JAR type
Update the Javadoc comment
Remove the Java task type tests from the e2e tests Since the original workflowJavaTaskE2ETest was used to test Java code types under Java task types, and this feature has been removed, this test is no longer needed.

type of FATJAR be51083587282062ceda775cd255c1f

type of NORMALJAR b06bccb0e0ebeaacd249a8000cf7a10

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:

(or)

Pull Request Notice

Pull Request Notice

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

fuchanghai commented 3 weeks ago

Please check if additional explanation is needed

ailiujiarui commented 2 weeks ago

Hi everyone, could you please let me know if there’s anything that needs improvement? Thanks!

sonarcloud[bot] commented 3 days ago

Please retry analysis of this Pull-Request directly on SonarCloud