Skip to content

Provide memory usage statistics for each commit #24

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sevanspowell
Copy link
Contributor

@sevanspowell sevanspowell commented May 12, 2021

  • Add a memory usage NixOS test that provides an overview of metadata-server's memory usage under high load.

Artifacts: https://hydra.iohk.io/build/6349761

Current total memory usage at ~19,529,266,024 bytes, with 99% of allocations happening in src/Cardano/Metadata/Store/Postgres.hs.

sevanspowell added a commit that referenced this pull request May 13, 2021
- Store metadata in the file system instead of a Postgres DB. PR #24
demonstrates that a large user of memory is the Postgres store,
specifically building the query for large batch-style requrests. This
could be investigated further, but a quick fix is simply to use a
file-based store. There is no need to build a query, and files can be
looked up quickly using their filename.
- Add a memory usage NixOS test that provides an overview of
  metadata-server's memory usage under high load. It doesn't actually
  "test" anything, only provides statistics.
@sevanspowell sevanspowell force-pushed the feature/ADP-869-provide-memory-usage-stats branch from 93308e7 to 7753b36 Compare May 18, 2021 04:44
@sevanspowell sevanspowell marked this pull request as ready for review May 18, 2021 04:46
@sevanspowell sevanspowell requested review from piotr-iohk and rvl May 18, 2021 04:46
sevanspowell added a commit that referenced this pull request May 18, 2021
- Store metadata in the file system instead of a Postgres DB. PR #24
demonstrates that a large user of memory is the Postgres store,
specifically building the query for large batch-style requrests. This
could be investigated further, but a quick fix is simply to use a
file-based store. There is no need to build a query, and files can be
looked up quickly using their filename.
sevanspowell added a commit that referenced this pull request May 18, 2021
- Store metadata in the file system instead of a Postgres DB. PR #24
demonstrates that a large user of memory is the Postgres store,
specifically building the query for large batch-style requrests. This
could be investigated further, but a quick fix is simply to use a
file-based store. There is no need to build a query, and files can be
looked up quickly using their filename.
sevanspowell added a commit that referenced this pull request May 18, 2021
- Store metadata in the file system instead of a Postgres DB. PR #24
demonstrates that a large user of memory is the Postgres store,
specifically building the query for large batch-style requrests. This
could be investigated further, but a quick fix is simply to use a
file-based store. There is no need to build a query, and files can be
looked up quickly using their filename.
sevanspowell added a commit that referenced this pull request May 19, 2021
- Store metadata in the file system instead of a Postgres DB. PR #24
demonstrates that a large user of memory is the Postgres store,
specifically building the query for large batch-style requrests. This
could be investigated further, but a quick fix is simply to use a
file-based store. There is no need to build a query, and files can be
looked up quickly using their filename.
sevanspowell added a commit that referenced this pull request May 25, 2021
- Store metadata in the file system instead of a Postgres DB. PR #24
demonstrates that a large user of memory is the Postgres store,
specifically building the query for large batch-style requrests. This
could be investigated further, but a quick fix is simply to use a
file-based store. There is no need to build a query, and files can be
looked up quickly using their filename.
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.

1 participant