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.

Sign up for GitHub

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

Jump to bottom

feat: dump-db command #65

Merged
merged 3 commits into from
Sep 29, 2023
Merged

feat: dump-db command #65

merged 3 commits into from
Sep 29, 2023

Conversation

mfw78
Copy link
Contributor

@mfw78 mfw78 commented Sep 28, 2023

Description

This PR adds a dump-db command feature to meet the requirements for analysing inter-watch-tower performance.

Changes

  • Add dump-db command.
  • Dry Registry code for loading orders.

How to test

  1. Test with no database (confirm create keys).
  2. Test with existing database (confirm loads keys correctly).

Related Issues

Fixes #48

@mfw78 mfw78 added enhancement New feature or request E:1.2: Watch Tower Service https://github.com/cowprotocol/pm/issues/8 labels Sep 28, 2023
@mfw78 mfw78 self-assigned this Sep 28, 2023
@mfw78 mfw78 changed the title Dump db dump-db command Sep 28, 2023
@mfw78 mfw78 linked an issue Sep 28, 2023 that may be closed by this pull request
feat: dump registry of chain #48
Closed
1 task
@mfw78 mfw78 changed the title dump-db command feat: dump-db command Sep 28, 2023
alfetopito
alfetopito approved these changes Sep 28, 2023
View reviewed changes
src/index.ts Show resolved Hide resolved
src/types/model.ts Outdated Show resolved Hide resolved
src/types/model.ts Outdated Show resolved Hide resolved
anxolin
anxolin approved these changes Sep 28, 2023
View reviewed changes
src/commands/dumpDb.ts Outdated Show resolved Hide resolved
src/index.ts Show resolved Hide resolved
src/types/model.ts Outdated Show resolved Hide resolved
@mfw78 mfw78 mentioned this pull request Sep 29, 2023
Merged
2 tasks
@mfw78 mfw78 changed the base branch from sdk-orderbook to standalone September 29, 2023 03:11
@mfw78 mfw78 merged commit f0491f8 into standalone Sep 29, 2023
@mfw78 mfw78 deleted the dump-db branch September 29, 2023 03:35
@github-actions github-actions bot locked and limited conversation to collaborators Sep 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@alfetopito alfetopito alfetopito approved these changes

@anxolin anxolin anxolin approved these changes

@shoom3301 shoom3301 Awaiting requested review from shoom3301

Assignees

@mfw78 mfw78

Labels
E:1.2: Watch Tower Service https://github.com/cowprotocol/pm/issues/8 enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

feat: dump registry of chain
3 participants
@mfw78 @alfetopito @anxolin