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.

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]: it takes 12s to return 'count(*)' query for an unloaded , empty collection #37532

Closed
1 task done
shanghaikid opened this issue Nov 8, 2024 · 7 comments
Closed
1 task done
Assignees
Labels
kind/bug Issues or changes related a bug priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Milestone

Comments

@shanghaikid
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Environment

- Milvus version: v2.4.14 & master

Current Behavior

it takes 12s to finish the query, which freezes the attu every time when loading the collection info.

2024-11-08T05:25:07.934Z] Milvus-sdk-node debug: [Request]v054ps.....>default>Query(15s): {"collection_name":"t","expr":"","output_fields":["count(*)"],"db_name":"default","query_params":[]}
[2024-11-08T05:25:19.963Z] Milvus-sdk-node debug: [Response(12029ms)]v054ps.....>default>Query: {"fields_data":[],"output_fields":[],"status":{"extra_info":{},"error_code":"UnexpectedError","reason":"failed to query: collection not loaded[collection=453759196215302711]","code":101,"retriable":false,"detail":"failed to query: collection not loaded[collection=453759196215302711]"},"collection_name":""}

Expected Behavior

it should return collection not loaded error right away.

Steps To Reproduce

No response

Milvus Log

No response

Anything else?

No response

@shanghaikid shanghaikid added kind/bug Issues or changes related a bug needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Nov 8, 2024
@yanliang567
Copy link
Contributor

/assign @bigsheeper
/unassign

@yanliang567 yanliang567 added triage/accepted Indicates an issue or PR is ready to be actively worked on. and removed needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Nov 8, 2024
@yanliang567 yanliang567 added this to the 2.5.0 milestone Nov 8, 2024
@yanliang567 yanliang567 added the priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. label Nov 8, 2024
@bigsheeper
Copy link
Contributor

Quick question: Does the 12s include the time for quick setup (which including CreateCollection and CreateIndex)? @shanghaikid

@shanghaikid
Copy link
Contributor Author

Quick question: Does the 12s include the time for quick setup (which including CreateCollection and CreateIndex)? @shanghaikid

no, just query.

@xiaofan-luan
Copy link
Collaborator

UnexpectedError

UnexpectedError ->
I guess the error code is hacked somewhere, cause retry some where.

And the timeout is not set for some reason, cause the retry take for ever.

@bigsheeper we need to fix the error code and make sure the retry follow timeout

@shanghaikid
Copy link
Contributor Author

@xiaofan-luan I believe this fix should also be applied to version 2.4.x. Otherwise, users of Attu with that version will face significant slowness.

@czs007
Copy link
Collaborator

czs007 commented Nov 14, 2024

@shanghaikid please help verify

@shanghaikid
Copy link
Contributor Author

verified on master-20241114-1d06d432-amd64

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Issues or changes related a bug priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Projects
None yet
Development

No branches or pull requests

6 participants