Union parameter not passed in multi-search queries
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 55/100
Rechercherichtung
Start at the MultiSearch.perform entry point shown in the issue and inspect how search_queries becomes the request body. Ensure the Union option is forwarded with the searches, then verify that multi-search requests using Union produce the expected grouped results.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
I tried to use multi-search with "Union" type on typesense==1.1.1.
Snippet:
for model, collection in model_and_collection:
query.append(
{
"collection": collection.schema_name,
"query_by": collection.query_by_fields,
"q": q,
}
)
results = client.multi_search.perform(
{
"union": True,
"searches": query,
}, {
"per_page": self.paginate_by,
"page": page_number,
}
)
But I realized that the result doesn't consider the union parameter. I have as many results as collection, and not a "grouped" hits results as expected.
After some investigation, I found out that the parameter "union" is not passed to the query.
def perform(
self,
search_queries: MultiSearchRequestSchema,
common_params: typing.Union[MultiSearchCommonParameters, None] = None,
) -> MultiSearchResponse:
"""
Perform a multi-search operation.
This method allows executing multiple search queries in a single API call.
It processes the search parameters, sends the request to the Typesense API,
and returns the multi-search response.
Args:
search_queries (MultiSearchRequestSchema):
A dictionary containing the list of search queries to perform.
The dictionary should have a 'searches' key with a list of search
parameter dictionaries.
common_params (Union[MultiSearchCommonParameters, None], optional):
Common parameters to apply to all search queries. Defaults to None.
Returns:
MultiSearchResponse:
The response from the multi-search operation, containing
the results of all search queries.
"""
stringified_search_params = [
stringify_search_params(search_params)
for search_params in search_queries.get("searches")
]
search_body = {"searches": stringified_search_params}
response: MultiSearchResponse = self.api_call.post(
MultiSearch.resource_path,
body=search_body,
params=common_params,
as_json=True,
entity_type=MultiSearchResponse,
)
return response
In the code above, you can see that "union" is not put into search_body.
Adding a line like that fixed it:
search_body["union"] = search_queries.get('union', False)
I will try to submit a MR about it.
- Vorherrschende Sprache
- Python
- Sterne
- 247
- Forks
- 66
- Ø Merge
- 22 Std. 47 Min.
- Gemergte PRs (30 T.)
- 8
Entwicklungsumgebung
Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus typesense/typesense-python
-
`MultiSearchResponse` type does not account for union search response shapeEvtl. vergeben @tharropoulos hat das vor 172 Tagen übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
typesense/typesense-python#128 ·
-
`documents.Documents.import_()` `batch_size` argument is ignored, type of `import_parameters` is missing `batch_size`Evtl. vergeben @tharropoulos hat das vor 245 Tagen übernommen. Offen
Schwierigkeit 2/5 1-2 Tage Anfängerfreundlichkeit 68/100
typesense/typesense-python#124 ·
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 68/100
typesense/typesense-python#100 ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100
typesense/typesense-python#122 · 3 Kommentare ·
-
[Bug] - DeprecationWarning on instantiation of ClientEvtl. vergeben @tharropoulos hat das vor 266 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 55/100
typesense/typesense-python#114 · 8 Kommentare ·
Alle Issues in typesense/typesense-python
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
Maintainer antworten meist innerhalb von 1 Tag
-
enhancement good first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
iceboundrock/AutoForge#304 ·
Maintainer antworten meist innerhalb von 1 Tag
-
enhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
fossasia/visdom-dev#77 ·
Maintainer antworten meist innerhalb von 4 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
aws-cloudformation/cfn-lint#4890 ·
Maintainer antworten meist innerhalb von 2 Tagen