Codecov Report

Merging #2955 into dev will increase coverage by 0.32%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                dev    #2955      +/-   ##
============================================
+ Coverage     38.13%   38.46%   +0.32%     
- Complexity     2620     2653      +33     
============================================
  Files           436      436              
  Lines         20084    20090       +6     
  Branches       2444     2444              
============================================
+ Hits           7660     7727      +67     
+ Misses        11714    11640      -74     
- Partials        710      723      +13     

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 284c50f...ea98a74. Read the comment docs.

@BoYiZhang Why is the default value of connecTimeout & socketTimeout set through front-end?
I think this is not a good solution, it's better set in backend.
Let's say, somebody create the dag through the Restful API without setting these two config ,then the API will throw exception.

@BoYiZhang Why is the default value of connecTimeout & socketTimeout set through front-end?
I think this is not a good solution, it's better set in backend.
Let's say, somebody create the dag through the Restful API without setting these two config ,then the API will throw exception.

Use (APACHE httpclient jar) to process HTTP requests

Default timeout: 60000ms

The front-end setting is mainly for display. If you do not need to set the timeout, the parameters are hidden

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities (and 0 Security Hotspots to review)
0 Code Smells

100.0% Coverage
0.0% Duplication

The version of Java (1.8.0_252) you have used to run this analysis is deprecated and we will stop accepting it from October 2020. Please update to at least Java 11.
Read more here

@BoYiZhang Why is the default value of connecTimeout & socketTimeout set through front-end?
I think this is not a good solution, it's better set in backend.
Let's say, somebody create the dag through the Restful API without setting these two config ,then the API will throw exception.

the related issue: #2882