apache / incubator-hugegraph-commons

(Archived Warning)Please visit "apache/hugegraph" repo instead
https://github.com/apache/hugegraph/tree/master/hugegraph-commons
Apache License 2.0
29 stars 45 forks source link

RestClient support to close idle connection periodically #26

Closed Linary closed 5 years ago

codecov[bot] commented 5 years ago

Codecov Report

Merging #26 into master will increase coverage by 1.34%. The diff coverage is 92.98%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #26      +/-   ##
============================================
+ Coverage     73.02%   74.36%   +1.34%     
- Complexity      442      457      +15     
============================================
  Files            44       44              
  Lines          1516     1533      +17     
  Branches        229      232       +3     
============================================
+ Hits           1107     1140      +33     
+ Misses          302      288      -14     
+ Partials        107      105       -2
Impacted Files Coverage Δ Complexity Δ
...ava/com/baidu/hugegraph/version/CommonVersion.java 50% <ø> (ø) 1 <0> (ø) :arrow_down:
...in/java/com/baidu/hugegraph/testutil/Whitebox.java 89.06% <100%> (+0.53%) 19 <0> (+2) :arrow_up:
...main/java/com/baidu/hugegraph/rest/RestClient.java 89.51% <92.3%> (+9.51%) 34 <17> (+10) :arrow_up:
.../main/java/com/baidu/hugegraph/event/EventHub.java 80.55% <0%> (-1.39%) 26% <0%> (-1%)
.../java/com/baidu/hugegraph/date/SafeDateFormat.java 80% <0%> (+80%) 4% <0%> (+4%) :arrow_up:

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 1a66a60...4b39e54. Read the comment docs.

javeme commented 5 years ago

The root cause is that the illegal connection is not closed. image