Brian Anderson
Brian Anderson
I'm putting this on the mvp, but it's probably ok to slip.
cc @mapleFU
Here's a string sorting experiment: https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=99ec4fd4c0e544282aa7094b718f3ddb They seem to be sorted by code point, then string length.
Use asciiflow like in project 4.
Some discussion here https://pingcap.slack.com/archives/CF31LFWP7/p1559943113093100
Thanks for the patch and sorry for not reviewing earlier @frobiac. I'd rather not add sleeps to fix broken tests. The correct way to fix this is to hook into...
@morenol thanks for the suggestion. On closer inspection, I see the tests are already filled with sleeps :( This is not a practice I want to endorse in this course!...
Per https://github.com/pingcap/talent-plan/issues/158, the CI should also be testing rustfmt and clippy.
There may be content in https://github.com/pingcap/talent-plan/issues/96 and linked issues to use for inspiration.
I am relatively convinced that failure is not the solution to error handling in Rust, but for the mvp we can leave it in and worry about it later.