Skip to content

esutil: wrap bulk item failures with response context#1498

Open
thani-ath-nain wants to merge 1 commit into
elastic:mainfrom
thani-ath-nain:fix/1428-pr2-bulk-indexer-item-error-context
Open

esutil: wrap bulk item failures with response context#1498
thani-ath-nain wants to merge 1 commit into
elastic:mainfrom
thani-ath-nain:fix/1428-pr2-bulk-indexer-item-error-context

Conversation

@thani-ath-nain

Copy link
Copy Markdown
Contributor

Summary

Part of #1428 (PR 2/3).

Adds response context to per-item bulk indexer failures so callers can inspect the failed item details via errors. As, while preserving existing callback flow and behavior.

What changed

  • Exported additive error type.
  • Updated the bulk flush item-failure path to pass the error into OnFailure.

Compatibility

  • No exported API changes.
  • No signature changes.
  • This is additive: consumers can opt in to richer inspection using errors.As(err, *BulkIndexerItemError).

Testing

make lint && make test-unit

@prodsecmachine

prodsecmachine commented May 31, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants