Brian Anderson

Results 400 comments of Brian Anderson

I'm putting this on the mvp, but it's probably ok to slip.

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.