Skip to content

Threadsafe cache#167

Open
ssande-fwd wants to merge 2 commits into
microsoft:masterfrom
ssande-fwd:threadsafe-cache
Open

Threadsafe cache#167
ssande-fwd wants to merge 2 commits into
microsoft:masterfrom
ssande-fwd:threadsafe-cache

Conversation

@ssande-fwd
Copy link
Copy Markdown
Contributor

Fixes #165

You can test using this program https://github.com/ssande-fwd/mkc-concurrency-test
You can see the error output I would get every 20 or so compilations in the readme for that program.

@ssande-fwd
Copy link
Copy Markdown
Contributor Author

@abchatra this is starting to become important for MicroChat. Can you have somebody review this? I would like to avoid creating a fork.

@abchatra abchatra requested a review from riknoll May 19, 2026 15:37
@ssande-fwd
Copy link
Copy Markdown
Contributor Author

I just ran an AI audit since it's been a while since I wrote this. Apparently this operation is redundant because renameAsync will overwrite an existing file, but I haven't tested.
if (await host().existsAsync(filePath)) { await host().unlinkAsync(filePath); }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Concurrent mkc builds occasionally fail due to shared cache

1 participant