Skip to content

[Feature][SPI] Add DolphinSPILoader#6729

Closed
ruanwenjun wants to merge 1 commit intoapache:devfrom
ruanwenjun:dev_wenjun_patch6612
Closed

[Feature][SPI] Add DolphinSPILoader#6729
ruanwenjun wants to merge 1 commit intoapache:devfrom
ruanwenjun:dev_wenjun_patch6612

Conversation

@ruanwenjun
Copy link
Member

related issue #6612

@kezhenxu94
Copy link
Member

I think #6712 has already simplified the development and usage of the SPI mechanism. I personally think that Java native SPI is already easy to use and the DolphinScheduler’s plugin mechanism is kind of over designed.

@ruanwenjun
Copy link
Member Author

I think #6712 has already simplified the development and usage of the SPI mechanism. I personally think that Java native SPI is already easy to use and the DolphinScheduler’s plugin mechanism is kind of over designed.

Great, I will close this pr.

@ruanwenjun ruanwenjun closed this Nov 7, 2021
@kezhenxu94
Copy link
Member

#6723 Is another optimization of the alert plugins. I’ll continue to optimize the task plugins and finally we can get rid of the over designed plugin loader and Java developer will find it familiar and comfortable to develop a new plugin. For users it’s also easy to use the plugins without pain

@ruanwenjun ruanwenjun deleted the dev_wenjun_patch6612 branch May 23, 2022 03:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants