Skip to content

Commit

Permalink
chore: delete MemoryDB (AztecProtocol#4241)
Browse files Browse the repository at this point in the history
This module is not being used anywhere and was just being kept around as
a reference. However this results in a maintenance burden as this shares
tests with the KV DB, which means we have to keep feature parity (see
e.g. AztecProtocol#4238).

The KV DB is in a good place already, and we can always look at the repo
history if we wish to bring this back, so the simplest path forward is
to remove this.
  • Loading branch information
nventuro authored Jan 26, 2024
1 parent b82e70c commit 9e6250a
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 317 deletions.
2 changes: 1 addition & 1 deletion yarn-project/end-to-end/src/e2e_persistence.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,7 @@ describe('Aztec persistence', () => {

expect(ownerBalance).toEqual(initialOwnerBalance - 500n);
expect(targetBalance).toEqual(500n);
});
}, 30_000);
});

describe.each([
Expand Down
1 change: 0 additions & 1 deletion yarn-project/pxe/src/database/index.ts
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
export * from './pxe_database.js';
export * from './memory_db.js';
67 changes: 0 additions & 67 deletions yarn-project/pxe/src/database/memory_db.test.ts

This file was deleted.

248 changes: 0 additions & 248 deletions yarn-project/pxe/src/database/memory_db.ts

This file was deleted.

0 comments on commit 9e6250a

Please sign in to comment.