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

Increase the number of blocks to keep stale messages #945

Merged

Conversation

MechanicalTyler
Copy link
Contributor

Related Github tickets

Background

We just increase our block speed drastically. As such, we need to increase the amount of blocks to allow messages to remain stale. This is because we have more blocks per minute.

We also don't need to check quite as often for the same reason.

Testing completed

  • test coverage exists or has been added/updated
  • tested in a private testnet

Breaking changes

  • I have checked my code for breaking changes
  • If there are breaking changes, there is a supporting migration.

We just increase our block speed drastically.  As such, we need to
increase the amount of blocks to allow messages to remain stale.
This is because we have more blocks per minute.

We also don't need to check quite as often for the same reason.
Copy link
Contributor

@byte-bandit byte-bandit left a comment

Choose a reason for hiding this comment

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

I would also be interested in performance metrics in general. There's a lot of stuff happening on every block, especially I see a lot of unoptimised/unbatched kv store reads/writes, and we just upped execution speed by 500% ish.

@taariq
Copy link
Contributor

taariq commented Aug 11, 2023

@byte-bandit any recommendations for monitoring and evaluating call performance?

@byte-bandit
Copy link
Contributor

@taariq Traditionally I'd use some tracing in each modules end block logic, with that telemetry being sent and processed on services like Datadog or Grafana.

With a decentralised system, maybe a monitoring module is the more idiomatic approach. Something that'll hook into the end block logic and track time, storing the results with each block on the chain so they're retrievable by anyone.

However, since anyone can run a node and we have little to no information what kind of hardware each node is running on, I don't think those results will be too reliable. Reproducible profiling as part of CI benchmarking would be better, but that's something for a full time devops role.

@taariq
Copy link
Contributor

taariq commented Aug 11, 2023

We will wait for the screams of block signing and jailing agony, then. Merging!

@taariq taariq merged commit 8ef8bd1 into palomachain:master Aug 11, 2023
@taariq taariq deleted the toaster/increase-stale-message-keep-time branch August 11, 2023 12:54
MechanicalTyler referenced this pull request in VolumeFi/paloma Aug 14, 2023
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