- Issue created by @ayrmax
- π¬π§United Kingdom scott_euser
Hi! Thanks for the contribution, can you make a merge request instead so tests run + easier for us maintaining to review/comment. Thank you!
- πͺπΈSpain gxleano CΓ‘ceres
Thanks @ayrmax and @scott_euser!
I've been testing the issue and it seems to be working as expected, so moving to RTBC.
- π¬π§United Kingdom scott_euser
I suppose its fine too; I searched all code that calls querySearch and they all seem to pass a limit so it probably has no other consequences.
I don't understand why the 16,384 limit passed already is not fine though as that seems to match Milvus limit documentation: https://milvus.io/docs/limitations.md - is it indicating that this is a symptom of a wider problem of us not handling limit/topk correctly in Milvus?