Skip to content

Add SearchFailure field in ByQueryResponse #1704

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.

Sign up for GitHub

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

Jump to bottom
Closed
ffaoudi opened this issue Feb 21, 2021 · 0 comments · Fixed by #1705
Closed

Add SearchFailure field in ByQueryResponse #1704

ffaoudi opened this issue Feb 21, 2021 · 0 comments · Fixed by #1705
Labels
type: enhancement A general enhancement

Comments

@ffaoudi
Copy link
Contributor

ffaoudi commented Feb 21, 2021

org.elasticsearch.index.reindex.BulkByScrollResponse returns a field named searchFailures but the org.springframework.data.elasticsearch.core.query.ByQueryResponse does not hold this value. We need this value to be added in the spring data elasticsearch
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Feb 21, 2021
@ffaoudi ffaoudi changed the title Return SearchFailure in ByQueryResponse Add SearchFailure field in ByQueryResponse Feb 21, 2021
@sothawo sothawo linked a pull request Feb 22, 2021 that will close this issue
Add SearchFailure field in ByQueryResponse #1705
Merged
5 tasks
sothawo pushed a commit that referenced this issue Feb 22, 2021
Add SearchFailure field in ByQueryResponse.
1c549b7 
Original Pull Request #1705
Closes #1704
@sothawo sothawo added type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Feb 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: enhancement A general enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add SearchFailure field in ByQueryResponse ffaoudi/spring-data-elasticsearch
3 participants
@ffaoudi @sothawo @spring-projects-issues