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

Compile test and daemon binary separately to avoid OOM in runner #3046

Merged
merged 2 commits into from
Oct 14, 2021

Conversation

yixinglu
Copy link
Contributor

Try to fix the OOM issue thrown in runners.

@yixinglu yixinglu added the ready-for-testing PR: ready for the CI test label Oct 13, 2021
@Sophie-Xie Sophie-Xie added this to the v2.7.0 milestone Oct 14, 2021
@yixinglu yixinglu force-pushed the fix-compile-oom branch 2 times, most recently from 639464e to 19be4e9 Compare October 14, 2021 06:08
@yixinglu yixinglu changed the title WIP: Compile test and daemon binary separately to avoid OOM in runner Compile test and daemon binary separately to avoid OOM in runner Oct 14, 2021
@yixinglu yixinglu modified the milestones: v2.7.0, v3.0.0 Oct 14, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #3046 (fe87711) into master (e1de5af) will increase coverage by 0.00%.
The diff coverage is 75.00%.

❗ Current head fe87711 differs from pull request most recent head b840467. Consider uploading reports for the commit b840467 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3046   +/-   ##
=======================================
  Coverage   84.24%   84.24%           
=======================================
  Files        1287     1287           
  Lines      115639   115640    +1     
=======================================
+ Hits        97415    97426   +11     
+ Misses      18224    18214   -10     
Impacted Files Coverage Δ
src/graph/service/GraphService.cpp 74.60% <0.00%> (-1.21%) ⬇️
src/clients/meta/MetaClient.cpp 73.52% <100.00%> (+0.01%) ⬆️
src/graph/executor/query/DataCollectExecutor.cpp 90.63% <100.00%> (ø)
src/meta/processors/parts/ListHostsProcessor.cpp 72.13% <100.00%> (+0.08%) ⬆️
src/graph/context/Result.cpp 85.00% <0.00%> (-9.45%) ⬇️
src/graph/executor/StorageAccessExecutor.h 63.63% <0.00%> (-5.20%) ⬇️
src/graph/planner/Planner.cpp 75.00% <0.00%> (-5.00%) ⬇️
...eta/processors/session/SessionManagerProcessor.cpp 74.52% <0.00%> (-4.46%) ⬇️
src/graph/session/GraphSessionManager.cpp 79.53% <0.00%> (-1.76%) ⬇️
src/kvstore/raftex/Host.cpp 63.37% <0.00%> (-1.75%) ⬇️
... and 21 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 e1ff44f...b840467. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants