David Barsky

Hmm, I'm not sure why the license check fails with ``` thread 'tidy::test' panicked at xtask/src/tidy.rs:182:9: different set of licenses! New Licenses: null Missing Licenses: ``` Similarly, I don't know...

For the license check, that should be fixed by https://github.com/salsa-rs/salsa/pull/601.

> Is there a summary of the differences between Salsa 2 and Salsa 3, for those of us who don't follow the news? Like, new Salsa and salsa-2022 or the...

From a _programming_ model standpoint, I wrote [this document](https://hackmd.io/@endsofthreads/Sk8jfPfjR) as a guide/reference. From a "what features are now possible" standpoint, new Salsa makes the following possible, or at the very...

I'm going to close this PR in favor of work I've been doing on a macro imitating the existing `query_group` macro: https://github.com/davidbarsky/db-ext-macro/. Few notes: 1. That macro live in upstream...

mtime isn't sufficient (I believe this is a long-standing bug in Cargo...), but I've found that setting `"rust-analyzer.files.watcher": "server"` helps a bunch with this. VS Code's default file watching behavior...

For some additional context from Zulip, the current single-threaded loading behavior is especially problematic on EdenFS, which _really_ wants tooling to send as many IO requests as possible in parallel.

> this is a separate issue that we should fix, that is wait with that until all files are loaded into memory. I did some rough, back-of-napkin benchmarking on this:...

I'll try and summarize my understanding of (the mostly agreed upon?) plan to accomplish this: - Finish @DropDemBits's branch that introduces a [`SyntaxEditor`](https://rust-lang.zulipchat.com/#narrow/stream/185405-t-compiler.2Frust-analyzer/topic/.60SyntaxEditor.60.20replacement.20for.20Mutable.20Syntax.20trees). - The remaining work on that branch...

> The interesting question is whether this would make sense for cargo itself to have? This might be me spending too much time with Buck, but I wanted some lightweight...