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

onSuccess function for bulk helper #2199

Merged
merged 6 commits into from
Apr 2, 2024
Merged

onSuccess function for bulk helper #2199

merged 6 commits into from
Apr 2, 2024

Conversation

JoshMock
Copy link
Member

@JoshMock JoshMock commented Apr 1, 2024

For #2090

Includes refactor of the tryBulk result processing code, to make iterating over bulk response data easier to understand.

For #2090

Includes refactor of the tryBulk result processing code, to make
iterating over bulk response data easier to understand.
@JoshMock JoshMock marked this pull request as ready for review April 2, 2024 16:51
@JoshMock JoshMock merged commit 4aa00e0 into main Apr 2, 2024
12 checks passed
@JoshMock JoshMock deleted the bulk-onsuccess branch April 2, 2024 19:38
github-actions bot pushed a commit that referenced this pull request Apr 2, 2024
* Bulk helper onSuccess callback

For #2090

Includes refactor of the tryBulk result processing code, to make
iterating over bulk response data easier to understand.

* Add onSuccess tests for each datasource type

* Cleanup, additional comments

* Add documentation for onSuccess callback

* Update changelog

* Drop link to 8.14 release notes.

Page not yet published, breaking docs build.

(cherry picked from commit 4aa00e0)
@JoshMock JoshMock added the apply-to-serverless Used by automation to generate a patch from a PR and apply it to elasticsearch-serverless-js label Apr 3, 2024
JoshMock added a commit to elastic/elasticsearch-serverless-js that referenced this pull request Apr 3, 2024
JoshMock added a commit to elastic/elasticsearch-serverless-js that referenced this pull request Apr 3, 2024
JoshMock added a commit to elastic/elasticsearch-serverless-js that referenced this pull request Apr 3, 2024
Applies patches from elastic/elasticsearch-js#2199 and elastic/elasticsearch-js#2027,
adding support for an onSuccess callback and fixing a bug that would cause the helper to
hang when the flushInterval was lower than the request timeout.

---------

Co-authored-by: JoshMock <160161+JoshMock@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
apply-to-serverless Used by automation to generate a patch from a PR and apply it to elasticsearch-serverless-js backport 8.14
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant