-
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
enhance: support null in text match index #37517
Conversation
@smellthemoon E2e jenkins job failed, comment |
@smellthemoon cpp-unit-test check failed, comment |
298dea8
to
7e71049
Compare
@smellthemoon cpp-unit-test check failed, comment |
@smellthemoon E2e jenkins job failed, comment |
/run-cpu-e2e |
rerun cpp-unit-test |
@smellthemoon E2e jenkins job failed, comment |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #37517 +/- ##
==========================================
+ Coverage 80.55% 80.56% +0.01%
==========================================
Files 1356 1356
Lines 190059 190122 +63
==========================================
+ Hits 153102 153172 +70
- Misses 31526 31536 +10
+ Partials 5431 5414 -17
|
/run-cpu-e2e |
@smellthemoon cpp-unit-test check failed, comment |
7e71049
to
6f9410f
Compare
@smellthemoon E2e jenkins job failed, comment |
/run-cpu-e2e |
@smellthemoon E2e jenkins job failed, comment |
/run-cpu-e2e |
@smellthemoon E2e jenkins job failed, comment |
/run-cpu-e2e |
@smellthemoon E2e jenkins job failed, comment |
6f9410f
to
a170057
Compare
@smellthemoon E2e jenkins job failed, comment |
@smellthemoon cpp-unit-test check failed, comment |
a170057
to
81dfff8
Compare
@smellthemoon E2e jenkins job failed, comment |
/run-cpu-e2e |
@smellthemoon cpp-unit-test check failed, comment |
@smellthemoon E2e jenkins job failed, comment |
Signed-off-by: lixinguo <xinguo.li@zilliz.com>
81dfff8
to
3875510
Compare
/lgtm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: congqixia, smellthemoon 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 |
#37508