Skip to content

[Fix-#11942] fix error occurred when copying workflow#12045

Closed
wendongdi wants to merge 5 commits intoapache:devfrom
wendongdi:dev
Closed

[Fix-#11942] fix error occurred when copying workflow#12045
wendongdi wants to merge 5 commits intoapache:devfrom
wendongdi:dev

Conversation

@wendongdi
Copy link
Copy Markdown
Contributor

@wendongdi wendongdi commented Sep 19, 2022

Purpose of the pull request

fix the bug #12027

Brief change log

set id to null before creating the target

Verify this pull request

manually

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Sep 20, 2022

Codecov Report

Merging #12045 (7423e85) into dev (cb4ef54) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                dev   #12045      +/-   ##
============================================
- Coverage     38.65%   38.63%   -0.02%     
+ Complexity     4032     4031       -1     
============================================
  Files          1004     1004              
  Lines         37437    37437              
  Branches       4262     4262              
============================================
- Hits          14471    14465       -6     
- Misses        21319    21326       +7     
+ Partials       1647     1646       -1     
Impacted Files Coverage Δ
...api/service/impl/ProcessDefinitionServiceImpl.java 31.69% <100.00%> (ø)
...eduler/server/worker/task/WorkerHeartBeatTask.java 69.38% <0.00%> (-8.17%) ⬇️
...erver/master/processor/queue/TaskEventService.java 75.00% <0.00%> (-5.36%) ⬇️
...r/plugin/task/sqoop/parameter/SqoopParameters.java 56.41% <0.00%> (+1.28%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@wendongdi
Copy link
Copy Markdown
Contributor Author

@SbloodyS @caishunfeng wait for viewing ~

@github-actions github-actions bot added the UI ui and front end related label Sep 21, 2022
@SbloodyS SbloodyS added the bug Something isn't working label Sep 21, 2022
@SbloodyS
Copy link
Copy Markdown
Member

Please associate the issue @wendongdi

@wendongdi
Copy link
Copy Markdown
Contributor Author

Please associate the issue @wendongdi

i added it.

@wendongdi
Copy link
Copy Markdown
Contributor Author

it seems mix other commits , i will close this PR and reopen an other one

@wendongdi wendongdi closed this Sep 21, 2022
@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

100.0% 100.0% Coverage
0.0% 0.0% Duplication

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend bug Something isn't working UI ui and front end related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants