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.

Sign up for GitHub

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

Jump to bottom

test: Add test cases about expr for null and default support #37121

Merged
merged 1 commit into from
Oct 25, 2024
Merged

test: Add test cases about expr for null and default support #37121

merged 1 commit into from
Oct 25, 2024

Conversation

binbinlv
Copy link
Contributor

issue: #36129

@sre-ci-robot sre-ci-robot added size/L Denotes a PR that changes 100-499 lines. area/test sig/testing labels Oct 24, 2024
@mergify mergify bot added dco-passed DCO check passed. kind/test labels Oct 24, 2024
@mergify Mergify
Copy link
Contributor

mergify bot commented Oct 24, 2024

@binbinlv cpp-unit-test check failed, comment rerun cpp-unit-test can trigger the job again.

@mergify Mergify
Copy link
Contributor

mergify bot commented Oct 24, 2024

@binbinlv E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

@mergify mergify bot added the ci-passed label Oct 24, 2024
@yanliang567
Copy link
Contributor

/lgtm
/approve

@sre-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: binbinlv, yanliang567

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sre-ci-robot sre-ci-robot merged commit c285853 into milvus-io:master Oct 25, 2024
6 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@smellthemoon smellthemoon Awaiting requested review from smellthemoon

@yanliang567 yanliang567 Awaiting requested review from yanliang567

@qixuan0212 qixuan0212 Awaiting requested review from qixuan0212

@zhuwenxing zhuwenxing Awaiting requested review from zhuwenxing

Assignees

@yanliang567 yanliang567

Labels
approved area/test ci-passed dco-passed DCO check passed. kind/test lgtm sig/testing size/L Denotes a PR that changes 100-499 lines.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@binbinlv @yanliang567 @sre-ci-robot