Skip to content
This repository has been archived by the owner on May 2, 2024. It is now read-only.

Create relay-meter-service.md #9

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

Conversation

adshmh
Copy link
Contributor

@adshmh adshmh commented Apr 5, 2023

This is a proposal for the first version of relay-meter functionality being delivered by TxDB service.

@adshmh adshmh requested a review from rem1niscence as a code owner April 5, 2023 16:50
@height
Copy link

height bot commented Apr 5, 2023

This pull request has been linked to 1 task:

💡Tip: Add "Close T-19745" to the pull request title or description, to a commit message, or in a comment to mark this task as "Done" when the pull request is merged.

@adshmh
Copy link
Contributor Author

adshmh commented Apr 5, 2023

Link T-19745

Signed-off-by: Arash Deshmeh <adshmh@gmail.com>
@adshmh adshmh force-pushed the relay-meter-replacement-v0 branch from d9f6b18 to f57d9e9 Compare April 5, 2023 18:52
Copy link
Contributor

Choose a reason for hiding this comment

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

One think to take into account is that the tx db service does not hold specific info regarding rate limiting, it holds data of all relays which then is aggregated to data warehouse in multiple ways, one of them being available for rate limiting. What I mean is that tx db service does not have special data structures specific for rate limiting, so all the code for this will be made in a way that this is only presented as a reporting service, which then can be consumed by services like rate limiter

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants