blockblaz / blockblaz/zeam

Improve process_slots logger parameter design and consider CachedState architecture

Open
#274 1 comment 0 reactions 1 assignee Claimed by @guha-rahul View on GitHub
enhancement
Dominant language
Zig
Stars
97
Forks
39
PR merge metrics
No merged PRs in 30d

Description

The `process_slots` function currently takes `logger: anytype` as a parameter, which feels awkward and reduces readability.

```zig
pub fn process_slots(self: *BeamState, allocator: Allocator, target_slot: Slot, logger: anytype) !void
```

## Proposed Solutions

### CachedState wrapper
- Create `CachedState` struct with `BeamState` and logger as properties
- Better performance for block processing
- Efficient justifications handling (flatten only when needed for SSZ ops)

## Context
here https://github.com/blockblaz/zeam/pull/260#discussion_r2402001864
https://github.com/blockblaz/zeam/pull/260#discussion_r2402253194

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.