-
Notifications
You must be signed in to change notification settings - Fork 3k
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
fix: [2.5] Search for pk using raw data to reduce the overhead caused by views #37203
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: xiaocai2333 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@xiaocai2333 E2e jenkins job failed, comment |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 2.5 #37203 +/- ##
==========================================
+ Coverage 80.84% 80.88% +0.03%
==========================================
Files 1322 1322
Lines 183234 183175 -59
==========================================
+ Hits 148143 148155 +12
+ Misses 29886 29821 -65
+ Partials 5205 5199 -6
|
@xiaocai2333 E2e jenkins job failed, comment |
@xiaocai2333 go-sdk check failed, comment |
@xiaocai2333 cpp-unit-test check failed, comment |
@xiaocai2333 go-sdk check failed, comment |
@xiaocai2333 E2e jenkins job failed, comment |
@xiaocai2333 cpp-unit-test check failed, comment |
ba970e8
to
006503c
Compare
@xiaocai2333 E2e jenkins job failed, comment |
006503c
to
2380a49
Compare
Signed-off-by: Cai Zhang <[email protected]>
2380a49
to
d47710e
Compare
@xiaocai2333 go-sdk check failed, comment |
rerun go-sdk |
@xiaocai2333 go-sdk check failed, comment |
close it becasue 2.5 branch will be recreated. |
issue: #37152
master pr: #37202