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.

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

feat: add visor processing stats table #96

Merged
merged 1 commit into from
Oct 14, 2020
Merged

Conversation

iand
Copy link
Contributor

@iand iand commented Oct 13, 2020

Add a task that writes summary stats to a new table visor_processing_stats.

For each tipset processor (message discovery, state change, chain economics) and
for each message processor (currently only gas outputs) and for each actor state
change it writes the following metrics:

  • total number of items that have been discovered for processing
  • total number of items that have been processed
  • total number of items that have been processed but reported an error
  • total number of items that are currently being processed
  • highest epoch that has been processed
  • highest epoch that has not been processed

Stats are written every minute and only a single global task is used.

Note: branched from #94

@iand iand force-pushed the iand/processing-stats branch 3 times, most recently from 98db1d9 to 20d01ea Compare October 13, 2020 14:10
Name: "processingstats-refresh-rate",
Aliases: []string{"psr"},
Value: 0,
Usage: "Refresh frequency for processing stats (0 = disables refresh)",
Copy link
Contributor

Choose a reason for hiding this comment

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

is this in seconds?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is a Go duration, so it takes values like 30s and 15m

Base automatically changed from iand/chain-economics to master October 14, 2020 09:21
@iand iand force-pushed the iand/processing-stats branch 2 times, most recently from 6e4f7ed to c50e7c3 Compare October 14, 2020 10:01
@iand
Copy link
Contributor Author

iand commented Oct 14, 2020

Creating the indexes takes a couple of minutes on a populated database.

Add a task that writes summary stats to a new table visor_processing_stats.

For each tipset processor (message discovery, state change, chain economics) and
for each message processor (currently only gas outputs) and for each actor state
change it writes the following metrics:

 - total number of items that have been discovered for processing
 - total number of items that have been processed
 - total number of items that have been processed but reported an error
 - total number of items that are currently being processed
 - highest epoch that has been processed
 - highest epoch that has not been processed

Stats are written every minute and only a single global task is used.
@iand iand merged commit 7644b3e into master Oct 14, 2020
@iand iand deleted the iand/processing-stats branch October 14, 2020 12:13
placer14 added a commit that referenced this pull request Oct 14, 2020
* origin/master:
  feat: Add historical indexer metrics (#92)
  fix(lens): Include dependencies needed for Repo Lens (#90)
  Move migration to schema version 10
  fix(migrations): migrations require version 0
  feat: add message gas economy processing
  perf(db): reduce batch size for chain history indexer (#105)
  fix: use hash index type for visor_processing_actors_code_idx (#106)
  feat: set application name in postgres connection (#104)
  feat: add visor processing stats table (#96)
  feat(task): add chain economics processing (#94)
  get actor name for both versions of specs-actors (#101)
placer14 added a commit that referenced this pull request Oct 14, 2020
…-view

* origin/master:
  feat: Add miner_sector_posts tracking of window posts (#74)
  feat: Add historical indexer metrics (#92)
  fix(lens): Include dependencies needed for Repo Lens (#90)
  Move migration to schema version 10
  fix(migrations): migrations require version 0
  feat: add message gas economy processing
  perf(db): reduce batch size for chain history indexer (#105)
  fix: use hash index type for visor_processing_actors_code_idx (#106)
  feat: set application name in postgres connection (#104)
  feat: add visor processing stats table (#96)
  feat(task): add chain economics processing (#94)
  get actor name for both versions of specs-actors (#101)
  fix: remove errgrp from UnindexedBlockData persist
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.

2 participants