Repository navigation
observability log lock striping - #304
Open
davidabram wants to merge 4 commits into
Open
davidabram wants to merge 4 commits into
davidabram wants to merge 4 commits into
Conversation
davidabram
added this pull request to stack #302
October 8, 2026 17:48
davidabram
force-pushed
the
fix-logging
branch
2 times, most recently
from
October 8, 2026 18:23
4f1bab6 to
0be74aa
Compare
davidabram
removed this pull request from stack #302
October 10, 2026 08:24
davidabram
added this pull request to stack #312
October 10, 2026 08:24
davidabram
force-pushed
the
fix-logging
branch
from
October 10, 2026 11:49
0be74aa to
ab3042b
Compare
Capture the fixed-stripe synchronization design, concurrency and fallback requirements, documentation updates, and validation work for bounded log-lock memory. Plan: observability-log-lock-striping Tasks: T01, T02, T03 Co-authored-by: SCE <sce@crocoder.dev>
Reduce lock-registry overhead by hashing log paths across a fixed set of static mutexes. Scope the primary lock so fallback writes occur only after it is released, while retaining retention cleanup and poisoned-lock behavior. Plan: observability-log-lock-striping, T01 Co-authored-by: SCE <sce@crocoder.dev>
Exercise the real log persistence paths under concurrent writers, independent logger instances, fallback failures, poisoned locks, retention, file permissions, and redaction. Isolate same-stripe fallback and lock poisoning in bounded child processes so regressions fail deterministically without contaminating shared test state. Replace the stripe-index truncating cast with a checked conversion for clippy without changing behavior. Plan: observability-log-lock-striping Task: T02 Co-authored-by: SCE <sce@crocoder.dev>
Document the fixed 64-stripe in-process locking model, fallback lock ordering, poisoning semantics, and absence of cross-process coordination. Record the completed acceptance criteria and validation evidence for the lock-striping work. Plan: observability-log-lock-striping (T03) Co-authored-by: SCE <sce@crocoder.dev>
davidabram
force-pushed
the
fix-logging
branch
from
October 10, 2026 12:27
ab3042b to
a94ca95
Compare
davidabram
removed this pull request from stack #312
October 10, 2026 22:50
davidabram
added this pull request to stack #320
October 10, 2026 22:50
This branch has not been deployed
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
1 participant
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.