Skip to content
This repository has been archived by the owner on Aug 16, 2023. It is now read-only.

Fix HNSW range search result wrong #1000

Merged

Conversation

cydrain
Copy link
Collaborator

@cydrain cydrain commented Jul 24, 2023

No description provided.

@mergify
Copy link

mergify bot commented Jul 24, 2023

@cydrain Please associate the related issue to the body of your Pull Request. (eg. “issue: #”)

@liliu-z
Copy link
Member

liliu-z commented Jul 24, 2023

/lgtm
/approve

@sre-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cydrain, liliu-z

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

@cydrain
Copy link
Collaborator Author

cydrain commented Jul 24, 2023

/kind improvement

src/common/range_util.cc Outdated Show resolved Hide resolved
@cydrain cydrain force-pushed the caiyd_fix_hnsw_range_search_wrong branch from ee60277 to a39d776 Compare July 24, 2023 11:29
@sre-ci-robot sre-ci-robot removed the lgtm label Jul 24, 2023
@liliu-z
Copy link
Member

liliu-z commented Jul 24, 2023

/lgtm

@sre-ci-robot sre-ci-robot merged commit e3fbe6e into milvus-io:main Jul 24, 2023
6 checks passed
@cydrain cydrain deleted the caiyd_fix_hnsw_range_search_wrong branch July 24, 2023 11:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants