Brian Anderson
Brian Anderson
From https://github.com/pingcap/talent-plan/pull/162#issuecomment-500175840: The current name of the course is "Practical Networked Applications in Rust", aka PNA Rust. The "networked" part of the course is relatively unimportant. I wonder if the...
The goals for this project still aren't clear, or they are not what they should be, or the content of the course so far doesn't support the goals. The [readme](https://github.com/brson/talent-training/blob/master/rust/README.md)...
Each project has its own test suite and working example projects. Add them to a single workspace and set up CI.
Let's just do this once for the mvp. When we get CI setup (https://github.com/pingcap/talent-plan/issues/105) then we can add tests for fmt and clippy.
Related to https://github.com/pingcap/talent-plan/issues/144. The `KvStore` name was created before the `KvsEngine` trait, but now it's inconsistent with the trait name and the `SledKvsEngine` impl name. This just makes things less...
I had an idea to present the entire set of course material in a single consistent site. At the moment though it is not worth doing though, mostly because all...
In particular, keys could be canonicalized in some way before doing any comparisons, and key ranges could be specced in different ways. I basically think that there should be no...
This is an obvious but understandable weakness, particularly while compaction happens non-concurrently. Need to see how the text describing the log files corresponds to the example. How open the text...
In https://github.com/pingcap/talent-plan/pull/138/files we are probably going to strongly suggest using the crossbeam skiplist. This crate isn't finished nor published yet though. Per @stjepang the code is done though. The only...
The first or second project asks logging to be introduced, but after that it's never again suggested things to log.