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.

Sign up for GitHub

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

Jump to bottom

Coprocessor read should not get started before the related MPPTask get scheduled. #4687

Open
windtalker opened this issue Apr 18, 2022 · 0 comments
Open

Coprocessor read should not get started before the related MPPTask get scheduled. #4687

windtalker opened this issue Apr 18, 2022 · 0 comments
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@windtalker
Copy link
Contributor

Enhancement

For MPPTask with table scan, if some regions are not in current TiFlash node, we use CoprocessorReader to pull the data from other TiFlash nodes. Current implementation will start coprocessor read immediately after CoprocessorReader is constructed. After #4044 we introduce a scheduler for MPP queries, a MPP task should only get started after it is scheduled, since CoprocessorReader is part of mpp task, we should make sure that the coprocessor read should not get started before related MPPTask get scheduled.
@windtalker windtalker added the type/enhancement The issue or PR belongs to an enhancement. label Apr 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@windtalker