[Fix-17396] Update old command key constants

[reele]

Purpose of the pull request

close #17396 .

Brief change log

update constants.

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 contains incompatible change, you should also add it to docs/docs/en/guide/upgrade/incompatible.md

[ruanwenjun]

It would be better if we can deserialize the task params to SubWorkflowParameters in method checkSubWorkflowDefinitionValid.

[reele]

It would be better if we can deserialize the task params to SubWorkflowParameters in method checkSubWorkflowDefinitionValid.

yes, i realized, i committed a new code.

should i revert CommandKeyConstants ?

[ruanwenjun]

It would be better if we can deserialize the task params to SubWorkflowParameters in method checkSubWorkflowDefinitionValid.

yes, i realized, i committed a new code.

should i revert CommandKeyConstants ?

No, you can do change on CommandKeyConstants if we still need to use this class, if we don't rely on this class, then we can remove it.

[reele]

It would be better if we can deserialize the task params to SubWorkflowParameters in method checkSubWorkflowDefinitionValid.

yes, i realized, i committed a new code.
should i revert CommandKeyConstants ?

No, you can do change on CommandKeyConstants if we still need to use this class, if we don't rely on this class, then we can remove it.

ok.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud