Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Reduce meta2 memory allocations #1998

Merged
merged 2 commits into from
Mar 6, 2020

Conversation

fvennetier
Copy link
Member

SUMMARY
  • Fix the --profile parameter of oio-bootstrap.py.
  • Track and fix unneeded memory allocations.
  • Get rid of unused code.
ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME
  • meta2
SDS VERSION
openio 5.3.0.0b1.dev21

@fvennetier fvennetier requested review from murlock and AymericDu March 5, 2020 18:04
@fvennetier fvennetier force-pushed the reduce-memory-allocations branch from f080905 to 69eacae Compare March 6, 2020 12:49
@codecov-io
Copy link

Codecov Report

Merging #1998 into 5.x will increase coverage by 0.04%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##              5.x    #1998      +/-   ##
==========================================
+ Coverage   79.19%   79.23%   +0.04%     
==========================================
  Files         391      391              
  Lines       74191    74185       -6     
  Branches     7040     7039       -1     
==========================================
+ Hits        58750    58770      +20     
+ Misses      15202    15172      -30     
- Partials      239      243       +4
Impacted Files Coverage Δ
meta2v2/generic.c 73% <100%> (-0.28%) ⬇️
meta2v2/meta2_utils.c 83.99% <100%> (-0.02%) ⬇️
tests/functional/m2_filters/test_filters.py 84.85% <0%> (-12.12%) ⬇️
tests/functional/container/test_replication.py 79.32% <0%> (-2.29%) ⬇️
sqliterepo/cache.c 84.53% <0%> (-1.19%) ⬇️
meta2v2/meta2_filters_action_container.c 63.36% <0%> (-1.02%) ⬇️
meta2v2/meta2_filters_extract.c 82.68% <0%> (-0.78%) ⬇️
oio/directory/admin.py 67.38% <0%> (-0.7%) ⬇️
rawx-apache2/src/rawx_repo_core.c 80.53% <0%> (-0.6%) ⬇️
core/cs.c 66.22% <0%> (-0.45%) ⬇️
... and 36 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 b386297...69eacae. Read the comment docs.

@fvennetier fvennetier merged commit ca92744 into open-io:5.x Mar 6, 2020
@fvennetier fvennetier deleted the reduce-memory-allocations branch March 9, 2020 10:37
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants