jonhoo / jonhoo/bystander

Moving from fast path to slow path on contention

Open
#3 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
30
Forks
3
PR merge metrics
No merged PRs in 30d

Description

Hey Jon, I’m completely new to rust, and watching your videos to get an understanding, and this one is probably over my head, but I think I spotted an issue in the implementation in the video and it’s still in the source and I didn’t want to leave a message on a year old video.

In the lines that run the fast path in the library, if there’s contention you do nothing and allow the fast path to retry. My understanding was, if there was contention, you’d check if it was over the threshold and then move to the slow path?

https://github.com/jonhoo/bystander/blob/bb66a3eff8364da63b7f6387bbef246df4762be1/src/lib.rs#L365

If you could clear that up for me, I would really appreciate it.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with src/lib.rs around line 365 and trace the fast-path behavior when contention occurs. Compare that behavior with the threshold and slow-path description in the issue, then determine the expected transition rule. The issue is complete only once the behavior is clarified and any agreed correction is covered by the relevant test.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
backend
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.