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

Exchange operator should check the schema before sending/receiver data #1500

Closed
windtalker opened this issue Mar 1, 2021 · 1 comment · Fixed by #1506
Closed

Exchange operator should check the schema before sending/receiver data #1500

windtalker opened this issue Mar 1, 2021 · 1 comment · Fixed by #1506
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@windtalker
Copy link
Contributor

windtalker commented Mar 1, 2021

Exchange operator is used to shuffle data in mpp, the schema of a exchange operator is generated in compile time based on DAG request, there is a chance that TiDB has some bugs so the data type is DAG request is not the same as the actually data type in runtime, which mean Exchange operator may send/receiver data that is not the same as its schema. We should check the schema and throw error if this happens.

@windtalker windtalker added the type/enhancement The issue or PR belongs to an enhancement. label Mar 1, 2021
@tisonkun
Copy link
Contributor

tisonkun commented Mar 1, 2021

related to #1492

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants