Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Interpreter & Executor & MPP unit test framework #4609

Closed
45 of 54 tasks
ywqzzy opened this issue Apr 7, 2022 · 1 comment
Closed
45 of 54 tasks

Interpreter & Executor & MPP unit test framework #4609

ywqzzy opened this issue Apr 7, 2022 · 1 comment
Assignees
Labels
component/compute component/coprocessor component/mpp good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. type/enhancement The issue or PR belongs to an enhancement. type/testing Issue or PR for testing

Comments

@ywqzzy
Copy link
Contributor

ywqzzy commented Apr 7, 2022

Enhancement

  1. Since there is no unit test for the Interpreter module, Interpreter test framework aims to supplement the content of the unit test, improve the stability of the module, and find more hidden bugs at home.
  2. TiFlash planner needs to rely on unit tests to ensure the quality and progress of development during the refactoring process.
  3. User can mock input columns and feed them into the "Execution streams(BlockInputStreams in TiFlash)", then they can test executors(except MPP related logic) of TiFlash locally.
  4. Further more, MPP related tests will be supported.
  5. When developing more features in the future, good unit testing can be used as the basis for development to further improve the efficiency and quality of code development.

Non MPP tests development progress:

MPP tests development progress:

Run with storage layer

Feel free to ask me anything about the Test Framework.

@ywqzzy ywqzzy added the type/enhancement The issue or PR belongs to an enhancement. label Apr 7, 2022
@ywqzzy ywqzzy self-assigned this Apr 7, 2022
ti-chi-bot pushed a commit that referenced this issue Apr 26, 2022
ti-chi-bot pushed a commit that referenced this issue May 25, 2022
@ywqzzy ywqzzy changed the title Interpreter test framework Interpreter & Executor test framework Jun 10, 2022
@ywqzzy ywqzzy added help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. type/testing Issue or PR for testing labels Jun 10, 2022
@ywqzzy ywqzzy mentioned this issue Sep 15, 2022
12 tasks
ti-chi-bot pushed a commit that referenced this issue Sep 15, 2022
ti-chi-bot pushed a commit that referenced this issue Sep 20, 2022
@SeaRise SeaRise mentioned this issue Oct 25, 2022
12 tasks
ti-chi-bot pushed a commit that referenced this issue Oct 25, 2022
ti-chi-bot pushed a commit that referenced this issue Oct 26, 2022
@ywqzzy ywqzzy mentioned this issue Dec 21, 2022
12 tasks
ti-chi-bot pushed a commit that referenced this issue Dec 22, 2022
ti-chi-bot pushed a commit that referenced this issue Dec 28, 2022
windtalker pushed a commit to windtalker/tiflash that referenced this issue Jan 4, 2023
JaySon-Huang pushed a commit to JaySon-Huang/tiflash that referenced this issue Jan 4, 2023
ti-chi-bot pushed a commit that referenced this issue Jan 5, 2023
ti-chi-bot pushed a commit that referenced this issue Mar 13, 2023
@ywqzzy ywqzzy mentioned this issue Apr 25, 2023
12 tasks
@SeaRise
Copy link
Contributor

SeaRise commented Jul 27, 2023

Looks like the job is done

@SeaRise SeaRise closed this as completed Jul 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/compute component/coprocessor component/mpp good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. type/enhancement The issue or PR belongs to an enhancement. type/testing Issue or PR for testing
Projects
None yet
Development

No branches or pull requests

4 participants