i am running a grouped query in the latest go sdk ...
# community-helpb
Bjarte Karlsen
03/13/2023, 8:20 AMi am running a grouped query in the latest go sdk and are seeing some strange numbers for out_of.
Fetched challenge candidates info found=1809 page=1 outOf=180291
i am fetching page1 there are 1809 total hits and 180k pages? this is my query
Copy code
params := &api.SearchCollectionParams{
Q: "*",
QueryBy: "owner",
FilterBy: StringPointer(fmt.Sprintf("number.play_id:%s", ids)),
IncludeFields: StringPointer("number.play_id,update_at,owner"),
GroupBy: StringPointer("owner"),
PerPage: IntegerPointer(250),
}Bjarte Karlsen
03/13/2023, 8:31 AMMaybe I can optimize my query as well, what i am looking for here is to fetch all owerns that owns all of the plays i send in. so if i send in 1,2,3 the owner has to atleast own 1 of each play_id to be a hit.
right now I am just doing the rest of the logic in golang, but doing it in lower layers is always better mechanical sympathy wise
k
Kishore Nallan
03/13/2023, 8:40 AMHave you tried making a raw curl request to see if that matches?
Kishore Nallan
03/13/2023, 8:40 AMOut of here means total number of results
b
Bjarte Karlsen
03/13/2023, 8:41 AMi have not tried raw curl no
Bjarte Karlsen
03/13/2023, 8:42 AMbut paged works with groups but outOf not?
k
Kishore Nallan
03/13/2023, 8:42 AMi.e.
out_ofb
Bjarte Karlsen
03/13/2023, 8:43 AMok it says “the total number of pages” in the comment
Bjarte Karlsen
03/13/2023, 8:43 AMso it looks like it is a documentation bug then
k
Kishore Nallan
03/13/2023, 8:43 AMMy bad, that's a typo error
Kishore Nallan
03/13/2023, 8:43 AMMust be a copy paste error
b
Bjarte Karlsen
03/13/2023, 8:43 AMdo you want me to file an issue?
k
Kishore Nallan
03/13/2023, 8:43 AMYes, thanks
b
Bjarte Karlsen
03/13/2023, 8:45 AMthe code is generated from openApi it looks like. what repo do you want the issue in? i do not think typesense-go is relevant?
k
Kishore Nallan
03/13/2023, 8:45 AMYes, the issue is here: https://github.com/typesense/typesense-api-spec/blob/master/openapi.yml#L1381
b
Bjarte Karlsen
03/13/2023, 9:01 AMBjarte Karlsen
03/13/2023, 9:01 AMwhy not just contribute and save you the work
k
Kishore Nallan
03/13/2023, 9:07 AMThank you! I will update and publish the go client with this change shortly.
b
Bjarte Karlsen
03/13/2023, 9:07 AMamazing!
Bjarte Karlsen
03/13/2023, 9:08 AMbtw have you considered adding a method to the typesnse-go client that just fetches all pages if things are paged?
k
Kishore Nallan
03/13/2023, 9:08 AMCan you elaborate what you mean by "all pages"?
b
Bjarte Karlsen
03/13/2023, 9:09 AMif the searchResult has more then one page then fetch the rest in a convenience method
Bjarte Karlsen
03/13/2023, 9:10 AMi have it now myself, so not a big one for me, just wondered
Bjarte Karlsen
03/13/2023, 9:10 AMmaybe it is best just to leave that code in userland
Bjarte Karlsen
03/13/2023, 9:11 AMi can see lots of different strategies for parallelization and error handling
k
Kishore Nallan
03/13/2023, 9:11 AMThe client is currently a very thin wrapper around the API but open to adding such convenience methods.
b
Bjarte Karlsen
03/13/2023, 9:11 AMok
Bjarte Karlsen
03/13/2023, 9:11 AMwill consider it if the use case emerges
👍 1