hiero-ledger / hiero-ledger/hiero-consensus-node

Revisit `maxPrecedingRecords` default in (modularized) `ConsensusConfig`

Open
#8,732 0 comments 0 reactions 1 assignee Claimed by @mhess-swl View on GitHub
Dominant language
Java
Stars
406
Forks
226
Avg merge
3d 4h
Merged PRs (30d)
210

Description

In PR #7983 we modified the `maxPrecedingRecords` default to be 950 instead of 3. However, there is a question of whether this new value is acceptable when there are only 1000 nanos (in theory) available for preceding records. We need to verify that either this value is acceptable and the app still works correctly, or that we need to change this config back to 3 and figure out a different way to create preceding genesis records.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.