Skip to content

Use file-based store instead of Postgres DB#29

Open
sevanspowell wants to merge 2 commits intomasterfrom
feature/ADP-869-fix-memory-leak
Open

Use file-based store instead of Postgres DB#29
sevanspowell wants to merge 2 commits intomasterfrom
feature/ADP-869-fix-memory-leak

Conversation

@sevanspowell
Copy link
Contributor

  • Store metadata in the file system instead of a Postgres DB.

    PR Provide memory usage statistics for each commit #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.
- 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 sevanspowell requested a review from rvl May 25, 2021 05:17
@sevanspowell
Copy link
Contributor Author

Artifacts here.

Current total memory usage at ~1,048,720,760 bytes, a ~18x decrease from existing levels.

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