We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
While tidb currently uses tree struct base executor, older versions of tidb and tispark still use list struct base executors.
The difference is that the list struct base executors does not have an executor_id and the list struct base executors is held by DAGRequest.executors.
tiflash/dbms/src/Flash/Coprocessor/DAGQueryBlock.cpp
Lines 170 to 232 in 0bafed9
Currently, ExecutorTestUtils only supports tree struct base executor, and we need to find a way to add UTs for list struct base executor.
related issue: #4609
The text was updated successfully, but these errors were encountered:
Add UT for list struct based executors (#5734)
927b52f
close #5534
Successfully merging a pull request may close this issue.
Enhancement
While tidb currently uses tree struct base executor, older versions of tidb and tispark still use list struct base executors.
The difference is that the list struct base executors does not have an executor_id and the list struct base executors is held by DAGRequest.executors.
tiflash/dbms/src/Flash/Coprocessor/DAGQueryBlock.cpp
Lines 170 to 232 in 0bafed9
Currently, ExecutorTestUtils only supports tree struct base executor, and we need to find a way to add UTs for list struct base executor.
related issue: #4609
The text was updated successfully, but these errors were encountered: