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

indexer: address async processor for address metrics #11972

Merged
merged 3 commits into from
May 17, 2023
Merged

indexer: address async processor for address metrics #11972

merged 3 commits into from
May 17, 2023

Conversation

gegaowp
Copy link
Contributor

@gegaowp gegaowp commented May 12, 2023 •
edited
Loading

Description

The related metrics are:

  • cumulative address count: address that ever sent or received a tx
  • cumulative active address count: address that ever sent a tx
  • daily active address: address that has sent tx over the previous day based on current checkpoint's timestamp

Test Plan

local run against testnet
Screenshot 2023-05-17 at 3 46 39 PM

@vercel
Copy link

vercel bot commented May 12, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
offline-signer-helper ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 17, 2023 9:59pm
4 Ignored Deployments
Name Status Preview Comments Updated (UTC)
explorer ⬜️ Ignored (Inspect) May 17, 2023 9:59pm
explorer-storybook ⬜️ Ignored (Inspect) May 17, 2023 9:59pm
sui-wallet-kit ⬜️ Ignored (Inspect) May 17, 2023 9:59pm
wallet-adapter ⬜️ Ignored (Inspect) May 17, 2023 9:59pm

@gegaowp gegaowp changed the title Addr async processor indexer: address async processor for address metrics May 17, 2023
@gegaowp gegaowp force-pushed the addr-async-processor branch from 497b061 to b43a821 Compare May 17, 2023 19:50
wlmyng
wlmyng approved these changes May 17, 2023
View reviewed changes
Copy link
Contributor

@wlmyng wlmyng left a comment

Choose a reason for hiding this comment

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

🆗

crates/sui-indexer/src/processors/processor_orchestrator.rs Outdated Show resolved Hide resolved
crates/sui-indexer/src/processors/processor_orchestrator.rs Outdated Show resolved Hide resolved
crates/sui-indexer/src/store/pg_indexer_store.rs Show resolved Hide resolved
@gegaowp gegaowp enabled auto-merge (squash) May 17, 2023 21:59
@gegaowp gegaowp merged commit 8a8b466 into MystenLabs:main May 17, 2023
@gegaowp gegaowp deleted the addr-async-processor branch May 17, 2023 23:36
gegaowp added a commit to gegaowp/sui that referenced this pull request May 18, 2023
@gegaowp
indexer: address async processor for address metrics (MystenLabs#11972)
9a29ddc 
## Description 

The related metrics are:
- cumulative address count: address that ever sent or received a tx
- cumulative active address count: address that ever sent a tx
- daily active address: address that has sent tx over the previous day
based on current checkpoint's timestamp

## Test Plan 

local run against testnet
![Screenshot 2023-05-17 at 3 46 39
PM](https://github.com/MystenLabs/sui/assets/106119108/6389bab3-e53e-4511-9099-8725f8306fc3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wlmyng wlmyng wlmyng approved these changes

@patrickkuo patrickkuo Awaiting requested review from patrickkuo

@healthydeve healthydeve Awaiting requested review from healthydeve

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gegaowp @wlmyng