Skip to content
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

Print heap dump error when oom #1093

Merged
merged 2 commits into from
Jul 15, 2020
Merged

Print heap dump error when oom #1093

merged 2 commits into from
Jul 15, 2020

Conversation

Linary
Copy link
Contributor

@Linary Linary commented Jul 13, 2020

Change-Id: I422a062530e931c8e5db54aeeae8592c0ab4d38c

Change-Id: I422a062530e931c8e5db54aeeae8592c0ab4d38c
@codecov
Copy link

codecov bot commented Jul 13, 2020

Codecov Report

Merging #1093 into master will increase coverage by 2.63%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1093      +/-   ##
============================================
+ Coverage     66.79%   69.42%   +2.63%     
- Complexity     5457     5584     +127     
============================================
  Files           331      331              
  Lines         26966    26966              
  Branches       3859     3859              
============================================
+ Hits          18012    18722     +710     
+ Misses         7186     6417     -769     
- Partials       1768     1827      +59     
Impacted Files Coverage Δ Complexity Δ
...a/com/baidu/hugegraph/backend/query/Condition.java 78.02% <0.00%> (-0.37%) 32.00% <0.00%> (ø%)
...m/baidu/hugegraph/backend/tx/GraphTransaction.java 80.75% <0.00%> (+0.33%) 296.00% <0.00%> (ø%)
...gegraph/backend/serializer/BinaryBackendEntry.java 82.27% <0.00%> (+1.26%) 28.00% <0.00%> (+1.00%)
...n/java/com/baidu/hugegraph/api/graph/BatchAPI.java 27.41% <0.00%> (+1.61%) 0.00% <0.00%> (ø%)
...du/hugegraph/backend/store/BackendSessionPool.java 76.38% <0.00%> (+2.77%) 17.00% <0.00%> (+1.00%)
...in/java/com/baidu/hugegraph/core/GraphManager.java 55.22% <0.00%> (+3.73%) 0.00% <0.00%> (ø%)
...aidu/hugegraph/backend/tx/AbstractTransaction.java 81.20% <0.00%> (+4.69%) 53.00% <0.00%> (ø%)
.../baidu/hugegraph/backend/store/BackendSession.java 80.95% <0.00%> (+4.76%) 1.00% <0.00%> (+1.00%)
.../com/baidu/hugegraph/backend/BackendException.java 69.23% <0.00%> (+15.38%) 5.00% <0.00%> (+1.00%)
...va/com/baidu/hugegraph/metrics/ServerReporter.java 93.10% <0.00%> (+20.68%) 0.00% <0.00%> (ø%)
... and 8 more

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 3c358f8...7aba522. Read the comment docs.

Change-Id: I43b38b9f4261720a075e77ed6b7fe0ccd67c24e3
@javeme javeme merged commit c1eea98 into master Jul 15, 2020
@javeme javeme deleted the dump-oom branch July 15, 2020 03:15
Linary added a commit that referenced this pull request Jul 20, 2020
Change-Id: I422a062530e931c8e5db54aeeae8592c0ab4d38c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants