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

Support check partition placement constraints when local transactions read data #21527

Closed
6 tasks
Yisaer opened this issue Dec 7, 2020 · 1 comment
Closed
6 tasks
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@Yisaer
Copy link
Contributor

Yisaer commented Dec 7, 2020

Development Task

Here is the detailed description: #21426 . This issue describe the implementation.

Implementation

To check the txn-scope constraint, we finally decide to check the txn-scope and the placement policy during executorBuilder.build function. If the executor read the data, the constraint should be checked during build.

Here are the Executor we are going to add checking logic:

  • TableReaderExecutor
  • IndexReaderExecutor
  • IndexLookUpReaderExecutor
  • IndexMergeReaderExecutor
  • PointGetExecutor
  • BatchPointGetExecutor
@Yisaer Yisaer added the type/enhancement The issue or PR belongs to an enhancement. label Dec 7, 2020
@Yisaer
Copy link
Contributor Author

Yisaer commented Dec 7, 2020

@XuHuaiyu @djshow832 PTAL

@Yisaer Yisaer closed this as completed Dec 10, 2020
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

No branches or pull requests

1 participant