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

Do not create index estimator objects for proxy collection objects #13506

Merged
merged 3 commits into from
Feb 8, 2021

Conversation

jsteemann
Copy link
Contributor

Scope & Purpose

Do no create index estimator objects for proxy collection objects on coordinators and DB servers. Proxy objects are created on coordinators and DB servers for all shards, and they also make index objects available. In order to reduce the memory usage by these objects, we don't create any index estimator objects for indexes in those proxy objects. Index estimators usually take several KB of memory each, so not creating them will pay out for higher numbers of collections/shards.

  • 💩 Bugfix (requires CHANGELOG entry)
  • 🍕 New feature (requires CHANGELOG entry, feature documentation and release notes)
  • 🔥 Performance improvement
  • 🔨 Refactoring/simplification
  • 📖 CHANGELOG entry made

Backports:

  • No backports required

Testing & Verification

  • This change is a trivial rework / code cleanup without any test coverage.
  • The behavior in this PR was manually tested

Link to Jenkins PR run:
http://172.16.10.101:8080/view/PR/job/arangodb-matrix-pr/13890/

Do no create index estimator objects for proxy collection objects on
coordinators and DB servers. Proxy objects are created on coordinator and
DB servers for all shards, and they also make index objects available. In
order to reduce the memory usage by these objects, we don't create any
index estimator objects for indexes in those proxy objects. Index estimators
usually take several KB of memory each, so not creating them will pay out
for higher numbers of collections/shards.
@jsteemann jsteemann added this to the devel milestone Feb 7, 2021
@jsteemann jsteemann requested a review from mchacki February 7, 2021 20:36
CHANGELOG Outdated Show resolved Hide resolved
@jsteemann
Copy link
Contributor Author

Tests blue

Copy link
Member

@mchacki mchacki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

SideNote:
As soon as we have better Monitoring of Ressource usage on those structs we can add tests around this.

arangod/RocksDBEngine/RocksDBEdgeIndex.cpp Show resolved Hide resolved
@jsteemann jsteemann merged commit ddbd8a8 into devel Feb 8, 2021
@jsteemann jsteemann deleted the bug-fix/dont-create-index-estimators-for-stubs branch February 8, 2021 17:19
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.

2 participants