talent-plan / tinysql

A course to build the SQL layer of a distributed database.
Apache License 2.0
1.74k stars 524 forks source link

Tracking issue for improve TinySQL as a learning-friendly mini distributed relational database #83

Open rebelice opened 3 years ago

rebelice commented 3 years ago

It is corresponding to the effort towards Talent Plan v3.0.

According to user feedback and my investigation, I found that TinySQL has serious issues. They make it a departure from the learning-friendly mini distributed relational database:

  1. Not mini. The TinySQL has more than 100,000 lines of code. It is almost a copy of TiDB, and then part of the code is deleted. It contains a lot of irrelevant code and design.
  2. Documents unfriendly. It almost only briefly explained the relevant knowledge topics and did not explain the project structure.
  3. Poor course design. The topics explained in each lab are very large, but the content that needs to be implemented is only a small part.
  4. Poor comments. They can't help understand the code.

In order to solve the above problems, I will redesign and implement TinySQL. The main improvements in the plan are as follows:

Issues

yanguwan commented 3 years ago

Thanks Rebelice. The topic list is fine as next wave of Talent Plan.

feitian124 commented 3 years ago

:+1: