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

observability metrics #23

Open

jkallogjeri wants to merge 4 commits into main from AN-3424/observability

jkallogjeri commented Aug 20, 2023

No description provided.

jkallogjeri added 4 commits

August 20, 2023 10:30


          blocks and transactions in progress

25b3982


          changed to swaps

3c581b9


          finished swaps/transactions and added dbt schedule

ff88aee


          finished failing tests

ccc9516

jkallogjeri requested a review from forgxyz

September 17, 2023 20:13

WHYTEWYLL commented Oct 17, 2023

The block observability model seems adequate.
However, for transactions, I think we'll need an external source because the block log doesn't provide the number of transactions per block. If im correct transactions aren't limited to just swaps; they can represent any type of event. Therefore, we should group all events by the block timestamp and compare that to the number of transactions within each block.

If that's not feasible, we may have to limit our assurance to the block itself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet