Skip to content

Conversation

@cprussin
Copy link
Collaborator

Summary

Use the RPC as the source of truth for the publisher list & number of active feeds for insights hub.

Rationale

Previously we were using clickhouse to drive this data, however clickhouse can be inaccurate or delayed for a number of reasons.

When I originally built this I misunderstood and thought publishers would only show up in the rpc if they had a data point for the last slot, but in fact publishers show up regardless of if they are actively publishing or not.

So this PR moves to the more accurate source of truth, the chain itself.

How has this been tested?

  • Current tests cover my changes
  • Added new tests
  • Manually tested the code

@cprussin cprussin requested a review from a team as a code owner October 26, 2025 00:00
@vercel
Copy link

vercel bot commented Oct 26, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
component-library Ready Ready Preview Comment Oct 27, 2025 6:52pm
insights Ready Ready Preview Comment Oct 27, 2025 6:52pm
5 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
api-reference Skipped Skipped Oct 27, 2025 6:52pm
developer-hub Skipped Skipped Oct 27, 2025 6:52pm
entropy-explorer Skipped Skipped Oct 27, 2025 6:52pm
proposals Skipped Skipped Oct 27, 2025 6:52pm
staking Skipped Skipped Oct 27, 2025 6:52pm

Copy link
Contributor

@benduran benduran left a comment

Choose a reason for hiding this comment

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

left mostly open Qs, but otherwise LGTM.

Alright then

@vercel vercel bot temporarily deployed to Preview – proposals October 27, 2025 18:50 Inactive
@vercel vercel bot temporarily deployed to Preview – staking October 27, 2025 18:50 Inactive
@vercel vercel bot temporarily deployed to Preview – entropy-explorer October 27, 2025 18:50 Inactive
@vercel vercel bot temporarily deployed to Preview – developer-hub October 27, 2025 18:50 Inactive
@vercel vercel bot temporarily deployed to Preview – api-reference October 27, 2025 18:50 Inactive
@cprussin cprussin merged commit c1f723c into main Oct 27, 2025
10 checks passed
@cprussin cprussin deleted the cprussin/use-rpc-for-publishers branch October 27, 2025 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants