Skip to content

[Bug] [dolphinscheduler-api] Data Analysis Service API responds with jackson deserialization exception#11858

Merged
kezhenxu94 merged 1 commit intoapache:devfrom
ouyangyewei429:issue/#11854
Sep 11, 2022
Merged

[Bug] [dolphinscheduler-api] Data Analysis Service API responds with jackson deserialization exception#11858
kezhenxu94 merged 1 commit intoapache:devfrom
ouyangyewei429:issue/#11854

Conversation

@ouyangyewei429
Copy link
Contributor

@ouyangyewei429 ouyangyewei429 commented Sep 8, 2022

fix #11854

@SbloodyS SbloodyS added bug Something isn't working backend labels Sep 8, 2022
@SbloodyS SbloodyS added this to the 3.0.1 milestone Sep 8, 2022
Copy link
Member

@fuchanghai fuchanghai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can

@ouyangyewei429
Copy link
Contributor Author

ouyangyewei429 commented Sep 8, 2022

@SbloodyS @caishunfeng @zhuangchong pls review, tks!

@codecov-commenter
Copy link

codecov-commenter commented Sep 8, 2022

Codecov Report

Merging #11858 (a6b68f1) into dev (e101a5c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                dev   #11858   +/-   ##
=========================================
  Coverage     38.53%   38.53%           
+ Complexity     4038     4036    -2     
=========================================
  Files           993      993           
  Lines         36716    36716           
  Branches       4273     4275    +2     
=========================================
  Hits          14150    14150           
+ Misses        20940    20939    -1     
- Partials       1626     1627    +1     
Impacted Files Coverage Δ
.../apache/dolphinscheduler/api/dto/TaskCountDto.java 94.44% <ø> (ø)
...erver/master/processor/queue/TaskEventService.java 75.00% <0.00%> (-5.36%) ⬇️
...r/plugin/task/sqoop/parameter/SqoopParameters.java 55.12% <0.00%> (-1.29%) ⬇️
...eduler/server/worker/task/WorkerHeartBeatTask.java 76.59% <0.00%> (+8.51%) ⬆️

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

@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 8, 2022

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 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Contributor

@caishunfeng caishunfeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@caishunfeng
Copy link
Contributor

I had rerun the fail CI.

Copy link
Member

@SbloodyS SbloodyS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@ouyangyewei429
Copy link
Contributor Author

ouyangyewei429 commented Sep 9, 2022

@caishunfeng @SbloodyS changes has been approved, could it be merged ?

@kezhenxu94 kezhenxu94 merged commit db46e47 into apache:dev Sep 11, 2022
@zhuangchong zhuangchong modified the milestones: 3.0.1, 3.1.0 Sep 16, 2022
xdu-chenrj pushed a commit to xdu-chenrj/dolphinscheduler that referenced this pull request Sep 17, 2022
@caishunfeng caishunfeng added the release cherry-pick Mark this issue/PR had cherry-pick for release version label Sep 19, 2022
caishunfeng pushed a commit that referenced this pull request Sep 19, 2022
xdu-chenrj pushed a commit to xdu-chenrj/dolphinscheduler that referenced this pull request Oct 13, 2022
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 release cherry-pick Mark this issue/PR had cherry-pick for release version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] [dolphinscheduler-api] Data Analysis Service API responds with jackson deserialization exception

7 participants