Skip to content

test: [E2e Refactor] use vector datatype instead of hard code dataype names #41497

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
Merged
merged 7 commits into from
Apr 25, 2025
Merged

test: [E2e Refactor] use vector datatype instead of hard code dataype names #41497

merged 7 commits into from
Apr 25, 2025

Conversation

yanliang567
Copy link
Contributor

related issue: #40698

  1. use vector datat types instead of hard code datatpe names
  2. update search pagination tests
  3. remove checking distances in search results checking, for knowhere customize the distances for different metrics and indexes. Now only assert the distances are sorted correct.

@yanliang567
Update binary pagination search assertion for more stable
f5076e0 
Signed-off-by: yanliang567 <yanliang.qiao@zilliz.com>
@yanliang567
Update vector data type with enum instead of string
86b80b7 
Signed-off-by: yanliang567 <yanliang.qiao@zilliz.com>
@yanliang567
Fix gen sparse field scheme issue
f49871c 
Signed-off-by: yanliang567 <yanliang.qiao@zilliz.com>
@yanliang567
Merge from master
c3ef6a0 
Signed-off-by: yanliang567 <yanliang.qiao@zilliz.com>
@yanliang567
Resolve conflicts
705a373 
Signed-off-by: yanliang567 <yanliang.qiao@zilliz.com>
@sre-ci-robot sre-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines. label Apr 24, 2025
@sre-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: 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

@mergify mergify bot added dco-passed DCO check passed. kind/test labels Apr 24, 2025
@mergify Mergify
Copy link
Contributor

mergify bot commented Apr 24, 2025

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

@yanliang567
Fix typo
573e605 
Signed-off-by: yanliang567 <yanliang.qiao@zilliz.com>
@mergify Mergify
Copy link
Contributor

mergify bot commented Apr 24, 2025

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

@yanliang567
Fix vector field name issue
5979e0b 
Signed-off-by: yanliang567 <yanliang.qiao@zilliz.com>
@mergify mergify bot added the ci-passed label Apr 24, 2025
@binbinlv
Copy link
Contributor

/lgtm

@sre-ci-robot sre-ci-robot merged commit 70b3117 into milvus-io:master Apr 25, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wangting0128 wangting0128 Awaiting requested review from wangting0128

@zhuwenxing zhuwenxing Awaiting requested review from zhuwenxing

Assignees

@binbinlv binbinlv

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

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