cockroachdb / cockroachdb/cockroach

kvserver: sep-raft-log: type safety for log and state engine

Open
#97,618 1 comment 0 reactions 1 assignee Claimed by @pav-kv View on GitHub
C-cleanup T-kv
Dominant language
Go
Stars
32.5k
Forks
4.1k
PR merge metrics
PR metrics pending

Description

With the introduction of the separate raft log, we have a number of methods that operate on an engine which is assumed to always be (say) raft log. It would be nice to leverage the type system to make it clear that some methods only ever operate on one kind of engine.

Perhaps extensions of `storage.Read{Writ,}er`, etc, could be introduced and used in appropriate locations.

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.