Conversation
…r multiple transitions which will be addressed in a immediate follow up PR
- Add AccountSharedData.fromAccount helper for cleaner conversions - Remove redundant readRange method in buffer_pool - Fix errdefer ordering in SlotConstants initialization - Add stake account validation comments for clarity - Update epoch transitions tests to use accounts_db.Two - Use PubkeyMap for node_id_to_vote_accounts consistency
- Follow Zig naming convention (lowercase for constants) - Remove unused tower_state.zig module
- Replace FullInflationFeatures struct with fullInflationFeaturesEnabled method - Clone reward_status when creating child slot state - Remove unused stake.zig module
- Flatten EpochRewardStatus to use optional partitioned_indices - Remove intermediate types: EpochRewardPhase, StartBlockHeightAndRewards - Move ValidatorRewards and RewardsForPartitioning to calculation.zig - Fix memory leak in buildUpdatedStakeReward
- Propagate OutOfMemory errors in builtin program migrations - Fix slot parameter for stateless builtin feature activation check - Propagate account reader errors instead of swallowing with catch null - Handle NoCreditsToRedeem specifically in stake vote rewards
- Add missing defer deinit calls for fetched accounts - Flatten nested conditionals with early returns - Remove unnecessary scope blocks - Add issue links for unimplemented TODOs
…count tests - Remove intermediate delegated_stakes map by accumulating stakes directly into vote accounts during iteration - Add VoteAccount.equals() method for proper account comparison - Re-enable previously commented-out test assertions - Restore compile-time test guard for initRandom
- Use @min/@max instead of nested ternary in getRewardDistributionNumBlocks - Flatten loop conditionals with early continue pattern - Fix memory cleanup by using defer instead of errdefer - Clarify TODO comment about account cloning
- Remove struct, use standalone functions instead - Rename file to hasher.zig
- Add initial HookContext for future rpc methods - Add slot tracking atomics and accounts DB for processed/confirmed/finalized states - Make Rooted.largest_rooted_slot thread-safe using std.atomic.Value - Update replay service to update slot states as blocks are processed
- Add initForTests helper for test initialization - Move advanceReplay, trackNewSlots, freezeCompletedSlots, bypassConsensus into ReplayState as methods
- Add magic_tracker to HookContext for leader schedule lookups - Add GetSlotLeader parameters and response structs - Implement getSlotLeader method using magic_tracker
- Add GetSlotLeaders parameters and response structs - Implement getSlotLeders method in HookContext
- Add genesis hash computation (SHA256 of raw file bytes) to GenesisConfig.init - Update GensisConfig.init to return InitResult containing both config and computed config hash - Add getGenesisHash RPC method using precomputed hash - Add tests verifying mainnet/testnet/devnet genesis hashes
- Adjust formatting in replay/service.zig - Adjust formatting in rpc/methods.zig - Remove unused builtin import from Rooted.zig
yewman
force-pushed
the
harnew/epoch-state-tracking
branch
from
c482a6d to
8a050a8
Compare
- Add gossip table and shred version to startGossip rpc hooks context - Add getClusterNodes RPC method to startGossip rpc hooks context - Add JSON serialization for SocketAddr and ClientVersion - Use native types instead of strings for RpcContactInfo fields
yewman
force-pushed
the
harnew/epoch-state-tracking
branch
5 times, most recently
from
28f730f to
f3844f8
Compare
yewman
force-pushed
the
harnew/epoch-state-tracking
branch
3 times, most recently
from
6bb797e to
e250f1f
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.