You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@dolphinscheduler.apache.org by GitBox <gi...@apache.org> on 2022/11/29 03:06:07 UTC
[GitHub] [dolphinscheduler] lordk911 opened a new issue, #13035: [Bug] [data-quality & master-server] wrong logic when master-server check data-quality result
lordk911 opened a new issue, #13035:
URL: https://github.com/apache/dolphinscheduler/issues/13035
### Search before asking
- [X] I had searched in the [issues](https://github.com/apache/dolphinscheduler/issues?q=is%3Aissue) and found no similar issues.
### What happened
I'm using 3.0.3-SNAPSHOT , I made a data-quality task for test , the wrong data have been write to hdfs , but at last the task was in success state.
I find that the master-server code : DataQualityResultOperator.java line 98 :
https://github.com/apache/dolphinscheduler/blob/a28e206166a10010d36a5dc0d9e2fd54f799678b/dolphinscheduler-master/src/main/java/org/apache/dolphinscheduler/server/utils/DataQualityResultOperator.java#L98
the isFailure method directly return the compare result
https://github.com/apache/dolphinscheduler/blob/a28e206166a10010d36a5dc0d9e2fd54f799678b/dolphinscheduler-master/src/main/java/org/apache/dolphinscheduler/server/utils/DataQualityResultOperator.java#L142
https://github.com/apache/dolphinscheduler/blob/a28e206166a10010d36a5dc0d9e2fd54f799678b/dolphinscheduler-master/src/main/java/org/apache/dolphinscheduler/server/utils/DataQualityResultOperator.java#L176
there need a "!" operater in the isFailure method.
this task should be faild in process :
<img width="1122" alt="image" src="https://user-images.githubusercontent.com/19989300/204428710-6a42fbb3-83b9-471d-9cfd-a9426b25cab0.png">
### What you expected to happen
correct the logic of the master when handle the data-quality result.
### How to reproduce
just have a test of data-quality task in process.
### Anything else
_No response_
### Version
3.0.x
### Are you willing to submit PR?
- [ ] Yes I am willing to submit a PR!
### Code of Conduct
- [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [dolphinscheduler] caishunfeng closed issue #13035: [Bug] [data-quality & master-server] wrong logic when master-server check data-quality result
Posted by GitBox <gi...@apache.org>.
caishunfeng closed issue #13035: [Bug] [data-quality & master-server] wrong logic when master-server check data-quality result
URL: https://github.com/apache/dolphinscheduler/issues/13035
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [dolphinscheduler] github-actions[bot] commented on issue #13035: [Bug] [data-quality & master-server] wrong logic when master-server check data-quality result
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on issue #13035:
URL: https://github.com/apache/dolphinscheduler/issues/13035#issuecomment-1330014539
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](https://s.apache.org/dolphinscheduler-slack) and send your question to channel `#troubleshooting`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [dolphinscheduler] lordk911 commented on issue #13035: [Bug] [data-quality & master-server] wrong logic when master-server check data-quality result
Posted by GitBox <gi...@apache.org>.
lordk911 commented on issue #13035:
URL: https://github.com/apache/dolphinscheduler/issues/13035#issuecomment-1330014407
@zixi0825
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [dolphinscheduler] jieguangzhou commented on issue #13035: [Bug] [data-quality & master-server] wrong logic when master-server check data-quality result
Posted by GitBox <gi...@apache.org>.
jieguangzhou commented on issue #13035:
URL: https://github.com/apache/dolphinscheduler/issues/13035#issuecomment-1330031751
Hey @lordk911 , can you help to fix
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org