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
distsql, executor: remove pctx from trectx #52496
Conversation
Signed-off-by: Yang Keao <yangkeao@chunibyo.icu>
Signed-off-by: Yang Keao <yangkeao@chunibyo.icu>
add006b
to
b330291
Compare
[LGTM Timeline notifier]Timeline:
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lcwangchao, XuHuaiyu, zimulala The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #52496 +/- ##
================================================
+ Coverage 72.2344% 75.1523% +2.9179%
================================================
Files 1469 1470 +1
Lines 427098 434565 +7467
================================================
+ Hits 308512 326586 +18074
+ Misses 99447 88266 -11181
- Partials 19139 19713 +574
Flags with carried forward coverage won't be shown. Click here to find out more.
|
What problem does this PR solve?
Issue Number: close #52495
Problem Summary:
We have splited
PlanContext
into some tiny contexts (BuildPBContext
andRangerContext
). Now it's time to remove thepctx
fromtableReaderExecutorContext
.What changed and how does it work?
pctx
fromtableReaderExecutorContext
.any
indistsql
intoMetaOnlyInfoSchema
(I don't know why it's originallyany
. Maybe it's used to avoid cyclic dependency?)Check List
Tests
Side effects
Documentation
Release note