-
Notifications
You must be signed in to change notification settings - Fork 2k
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] Missing toString() method in many requests classes #12144
Comments
8 tasks
8 tasks
peteralfonsi
pushed a commit
to peteralfonsi/OpenSearch
that referenced
this issue
Mar 1, 2024
…Request classes. (opensearch-project#12163) fix opensearch-project#12144 Signed-off-by: Guillaume Alvarez <7364145+guillaume-alvarez@users.noreply.github.com>
rayshrey
pushed a commit
to rayshrey/OpenSearch
that referenced
this issue
Mar 18, 2024
…Request classes. (opensearch-project#12163) fix opensearch-project#12144 Signed-off-by: Guillaume Alvarez <7364145+guillaume-alvarez@users.noreply.github.com>
shiv0408
pushed a commit
to Gaurav614/OpenSearch
that referenced
this issue
Apr 25, 2024
…Request classes. (opensearch-project#12163) fix opensearch-project#12144 Signed-off-by: Guillaume Alvarez <7364145+guillaume-alvarez@users.noreply.github.com> Signed-off-by: Shivansh Arora <hishiv@amazon.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
Whereas SearchRequest has a toString() method, making debugging much easier, it is not present in all other requests.
For instance it is missing in
Related component
Clients
To Reproduce
Expected behavior
{requests=[SearchRequest{...},...]}
Additional Details
Plugins
N/A
Screenshots
N/A
Host/Environment (please complete the following information):
Additional context
I will try to provide a fix in PR.
The text was updated successfully, but these errors were encountered: