dolphinscheduler icon indicating copy to clipboard operation
dolphinscheduler copied to clipboard

[DSIP-72][Dynamic Task] Suggest remove dynamic task type

Open SbloodyS opened this issue 1 year ago • 12 comments

Search before asking

  • [X] I had searched in the DSIP and found no similar DSIP.

Motivation

The dynamic task type is currently supported in a user-configured manner. The user executes a while like loop to determine the exit condition. After the exit, specify multiple times of sub-workflows based on exit conditions.

The current task type has a lot of bugs, and I think the operation of executing multiple times of sub-workflows is not universal, and it is too customized for a certain business scenario. And at the same time, I search in the issue list found that only a single digit of users were using it.

So I suggest remove this dynamic task type.

Design Detail

No response

Compatibility, Deprecation, and Migration Plan

No response

Test Plan

No response

Code of Conduct

SbloodyS avatar Sep 05 '24 07:09 SbloodyS

+1

caishunfeng avatar Sep 05 '24 07:09 caishunfeng

+1

EricGao888 avatar Sep 05 '24 09:09 EricGao888

+1, and because of this issue try to remove an existing plugin for our product, and as I remember we do not remove anyone before. can we add some docs and guide someone who wants to remove other plugins? Such as we have to discuss in issue or mail thread, or we only remove plugin in major or minor version?

zhongjiajie avatar Sep 05 '24 13:09 zhongjiajie

+1, and because of this issue try to remove an existing plugin for our product, and as I remember we do not remove anyone before. can we add some docs and guide someone who wants to remove other plugins? Such as we have to discuss in issue or mail thread, or we only remove plugin in major or minor version?

I think we should remove plugin only in major version.

SbloodyS avatar Sep 05 '24 13:09 SbloodyS

+1, and because of this issue try to remove an existing plugin for our product, and as I remember we do not remove anyone before. can we add some docs and guide someone who wants to remove other plugins? Such as we have to discuss in issue or mail thread, or we only remove plugin in major or minor version?

I think we should remove plugin only in major version.

Agree with that, and we can add some deprecated notice for those task we remove in dev branch but still not released, WDYT

zhongjiajie avatar Sep 06 '24 02:09 zhongjiajie

Agree with that, and we can add some deprecated notice for those task we remove in dev branch but still not released, WDYT

Yes, we've already add it to the docs in https://github.com/apache/dolphinscheduler/blob/dev/docs/docs/en/guide/upgrade/incompatible.md

SbloodyS avatar Sep 06 '24 02:09 SbloodyS

+1

ruanwenjun avatar Sep 06 '24 03:09 ruanwenjun

@SbloodyS Hi SbloodyS. Can I understand that if a feature is found to have some bugs and has few users, then it can be removed?

lenboo avatar Sep 24 '24 01:09 lenboo

@SbloodyS Hi SbloodyS. Can I understand that if a feature is found to have some bugs and has few users, then it can be removed?

I think no PMC/Committer is willing to continue to maintain, and at the same time, there are bugs that are not fixed and rarely used. Then we will remove it. The situation is similar to that of #16093.

SbloodyS avatar Sep 24 '24 01:09 SbloodyS

That means if someone is willing to fix these bugs and the fixes pass verification, then the feature can be retained and not removed.

lenboo avatar Sep 24 '24 01:09 lenboo

That means if someone is willing to fix these bugs and the fixes pass verification, then the feature can be retained and not removed.

Yes.

SbloodyS avatar Sep 24 '24 01:09 SbloodyS

I will look for someone in the community who is willing to fix these bugs.

lenboo avatar Sep 24 '24 03:09 lenboo