IntersectMBO / IntersectMBO/lsm-tree

Make `RefCtx` less ubiquitous internally

Open
#793 0 comments 0 reactions 0 assignees View on GitHub
enhancement lsm-tree priority: low
Dominant language
Haskell
Stars
53
Forks
17
Avg merge
2d 3h
Merged PRs (30d)
4

Description

Since #781, sessions internally store a `RefCtx` and we pass it around for debugging purposes, but this makes `RefCtx` rather ubiquitous. We might be able to remove `RefCtx` as an argument to internal functions in places. For example, we could:

* Also store the `RefCtx` in structures closer to where they are needed, and/or
* Extract `RefCtx`'s on demand from `Ref`s that are already in scope rather than passing in `RefCtx` from outside, and/or
* Bundle `RefCtx` with other ubiquitous arguments

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.