-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[Bug]: When building an index for the same data, the CPU usage of the master image is 100%, but that of the 2.4 image is only 60% #39090
Comments
/unassign |
@ThreadDao In standalone mode, the default CPU usage is limited to 75%, so the usage of 2.4 is expected. However, I'm still investigating why the master can utilize 100% of the CPU. |
is there any config for this cpu usage limitation? I remember it was 50%? @xiaocai2333 |
yeah, the index-building pool size is usually set to 50% in standalone mode.
In standalone mode, the component that starts later will overwrite the corresponding configuration. Typically, the querynode starts last, so the index-building pool size is usually set to 50% in standalone mode. |
The implementation of hnsw was switched from hnswlib to faiss when the milvus version switch from 2.4 to master. It is possible that the omp setter does not work in faiss_hnsw train and add function ?/assign @alexanderguzhva |
https://github.com/zilliztech/knowhere/blob/main/src/index/hnsw/faiss_hnsw.cc#L113 From the test results, build_pool is effective and can limit the CPU resources used to build the index; the code also shows that it is limited by omp https://github.com/zilliztech/knowhere/blob/main/src/index/hnsw/faiss_hnsw.cc#L113 |
issue: #39090 pr: #39340 Signed-off-by: xianliang.li <[email protected]>
issue: #39090 The num_build_thread parameter will limit the number of build omps. This parameter will override the effect of buildIndexThreadPoolRatio. Removing this parameter will have no actual effect. This parameter is actually only used in the growing index, where it will be explicitly set. Signed-off-by: xianliang.li <[email protected]>
verified on 2.5-20250120-44649664-amd64 which cpu usage keeps on 50% in standalone mode. |
Is there an existing issue for this?
Environment
Current Behavior
Test index cost for sift 10m data on master and 2.4 branch. The cpu usage of master is 100% while the 2.4 is only 60%
metrics of stats-24-op-55-4263
pyroscope of stats-24-op-55-4263
metrics of stats-master-op-62-8296
pyroscope of stats-master-op-62-8296
Expected Behavior
No response
Steps To Reproduce
https://argo-workflows.zilliz.cc/archived-workflows/qa/8a74abd1-0140-4ce5-af21-f146e2275f1f?nodeId=zong-stats-index-4
Milvus Log
Anything else?
No response
The text was updated successfully, but these errors were encountered: