-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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: erase pk empty check when pk index replace raw data #30432
Conversation
@zhagnlu E2e jenkins job failed, comment |
/run-cpu-e2e |
52722f6
to
7a62fd8
Compare
@zhagnlu E2e jenkins job failed, comment |
rerun ut |
7a62fd8
to
22e3923
Compare
is this a problem for 2.3? |
22e3923
to
9e54dad
Compare
rerun ut |
@zhagnlu ut workflow job failed, comment |
yes, will cherry-pick to 2,.3 |
rerun ut |
6b8dc69
to
3f035bf
Compare
@zhagnlu E2e jenkins job failed, comment |
Signed-off-by: luzhang <luzhang@zilliz.com>
3f035bf
to
3b8a3d4
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #30432 +/- ##
==========================================
+ Coverage 80.23% 80.25% +0.01%
==========================================
Files 965 955 -10
Lines 136070 136151 +81
==========================================
+ Hits 109180 109267 +87
+ Misses 23171 23163 -8
- Partials 3719 3721 +2
|
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: czs007, zhagnlu 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 |
@zhagnlu please pick to 2.3 |
…0432) Signed-off-by: luzhang <luzhang@zilliz.com> Co-authored-by: luzhang <luzhang@zilliz.com>
#30350