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] SearchRequest is not serializing the Scroll time parameter #676

Open
frivard-coveo opened this issue Oct 19, 2023 · 1 comment
Open
Labels
enhancement New feature or request

Comments

@frivard-coveo
Copy link

What is the bug?

The Scroll API is not working properly because the ScrollId returned in the SearchResponse has an ill-defined validity.
The root cause appears to be that the SearchRequest Scroll parameter (or type Time) is not serialized, and thus the search validity cannot be specified.

How can one reproduce the bug?

Create a SearchRequest, and specify a Scroll with value "5m", which is 5 minutes.
With the search response, use the scroll id to do a ScrollRequest;
most of the time the server replies with "all shards failed".

What is the expected behavior?

The search should success with 0 or more results.

What is your host/environment?

Observed on Alpine Linux (hosted on AWS) and Windows 11 (developer laptop).

Do you have any screenshots?

No, but I have application logs (from our in-house application)

Do you have any additional context?

no.

@dblock
Copy link
Member

dblock commented Dec 4, 2023

@frivard-coveo Still want to try to fix this? I renamed this into a feature request to serialize the additional types.

@dblock dblock added enhancement New feature or request and removed untriaged bug Something isn't working labels Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants