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

New elasticsearch_v8 do not retry anymore #3175

Closed
defgenx opened this issue Feb 10, 2025 · 2 comments
Closed

New elasticsearch_v8 do not retry anymore #3175

defgenx opened this issue Feb 10, 2025 · 2 comments

Comments

@defgenx
Copy link
Contributor

defgenx commented Feb 10, 2025

Hello Redpanda team.

It looks like that the new Elasticsearch output has removed the retry mechanism since this PR (#3160). Is that normal ? Can we re-add it ?

I also see that the he upsert logic changed to rely on upsert key instead of using doc_as_upsert as before. Why this change ? Now we have to duplicate in .doc and in .upsert to achieve the same behavior ?
I did some tests and it looks like something is not working as expected.
I'll dig and try to provide more information soon.

@defgenx defgenx changed the title New elasticsearch_v8 output does not have retry anymore New elasticsearch_v8 output has some errors and do not retry anymore Feb 10, 2025
@defgenx defgenx changed the title New elasticsearch_v8 output has some errors and do not retry anymore New elasticsearch_v8 do not retry anymore Feb 10, 2025
@defgenx
Copy link
Contributor Author

defgenx commented Feb 11, 2025

Ok so I confirm in case of upsert we have to set both doc and upsert fields... Most of the time we want to doc_as_upsertso I think we should add this option. I can create a PR for this one.

@defgenx
Copy link
Contributor Author

defgenx commented Feb 12, 2025

Just closing the issue and will open a PR if needed.

@defgenx defgenx closed this as completed Feb 12, 2025
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

No branches or pull requests

1 participant