Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Bug] [dolphinscheduler-server] param lost when execute failed task #9595

Closed
2 of 3 tasks
zwZjut opened this issue Apr 19, 2022 · 8 comments
Closed
2 of 3 tasks

[Bug] [dolphinscheduler-server] param lost when execute failed task #9595

zwZjut opened this issue Apr 19, 2022 · 8 comments
Assignees
Labels
backend bug Something isn't working
Milestone

Comments

@zwZjut
Copy link
Contributor

zwZjut commented Apr 19, 2022

Search before asking

  • I had searched in the issues and found no similar issues.

What happened

param lost when execute failed task
branch 2.0.5 && dev

What you expected to happen

param will not lost when execute failed task

How to reproduce

create processDefinition test
create task A set parameter a and set OUT
create task B dependent task A and use parameter a; do something let task B 100% failed
run processDefinition test and server will create processInstance test_xxxx
find test_xxxx in list and execute failed tasks, when it is failed, open the task log :
"setSqlParamsMap: No Property with paramName: a is found in paramsPropsMap of task instance with id"

Anything else

No response

Version

2.0.5

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@zwZjut zwZjut added bug Something isn't working Waiting for reply Waiting for reply labels Apr 19, 2022
@github-actions
Copy link

Thank you for your feedback, we have received your issue, Please wait patiently for a reply.

  • In order for us to understand your request as soon as possible, please provide detailed information、version or pictures.
  • If you haven't received a reply for a long time, you can join our slack and send your question to channel #troubleshooting

@songjianet songjianet added backend and removed Waiting for reply Waiting for reply labels Apr 19, 2022
@caishunfeng caishunfeng added this to the 2.0.6-release milestone Apr 20, 2022
@Tianqi-Dotes Tianqi-Dotes self-assigned this Apr 20, 2022
@Tianqi-Dotes
Copy link
Member

@zwZjut hi, I cannot reproduce, could you provide your process definition script here?
I passed a param from task A to task B and set task B timeout failure, but in my test, task B can reach the param value from task A.

@zwZjut
Copy link
Contributor Author

zwZjut commented Apr 20, 2022

pic_1
pic_2
pic_3
pic_4
pic_5

if you still cannot reproduce, connect me in wechat by name of HY in ds group

@zwZjut
Copy link
Contributor Author

zwZjut commented Apr 20, 2022

you can also see difference between two tasks in meta data

@Tianqi-Dotes
Copy link
Member

dev part code is fine, you can try dev

@zwZjut
Copy link
Contributor Author

zwZjut commented Apr 22, 2022

dev part code is fine, you can try dev

will you cherry-pick to 2.0.6 ? is it easy to fix ?

@github-actions
Copy link

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

@github-actions github-actions bot added the Stale label May 23, 2022
@zwZjut
Copy link
Contributor Author

zwZjut commented May 23, 2022

this problem exists in 2.0.x

@github-actions github-actions bot removed the Stale label May 24, 2022
zwZjut pushed a commit to zwZjut/dolphinscheduler that referenced this issue Jun 28, 2022
caishunfeng pushed a commit that referenced this issue Jun 28, 2022
Co-authored-by: 宏豁 <honghuo.zw@alibaba-inc.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
backend bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants