-
Notifications
You must be signed in to change notification settings - Fork 4.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Improvement][API][num-8] add ProcessTaskRelationServiceImpl.createProcessTaskRelation #6970
Conversation
… dev_api2 merge from upstream dev
Codecov Report
@@ Coverage Diff @@
## dev #6970 +/- ##
============================================
- Coverage 32.90% 32.80% -0.11%
+ Complexity 1625 1620 -5
============================================
Files 432 432
Lines 14379 14379
Branches 1446 1446
============================================
- Hits 4732 4717 -15
- Misses 9191 9204 +13
- Partials 456 458 +2
Continue to review full report at Codecov.
|
SonarCloud Quality Gate failed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
} | ||
int insert = processTaskRelationMapper.batchInsert(processTaskRelationLogs); | ||
int insertLog = processTaskRelationLogMapper.batchInsert(processTaskRelationLogs); | ||
if ((insert & insertLog) > 0) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe this judge condition is not always correct, such as 1&2=0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this scenario occurs, there is a problem
#6772
Adding tasks to workflow, adding upstream dependencies, and adding downstream dependencies