Skip to content

Add retry mechanism for elastic database#984

Merged
bogdanfazakas merged 4 commits into
mainfrom
fix/add-elastic-retry-mech
Jul 18, 2025
Merged

Add retry mechanism for elastic database#984
bogdanfazakas merged 4 commits into
mainfrom
fix/add-elastic-retry-mech

Conversation

@bogdanfazakas

@bogdanfazakas bogdanfazakas commented Jul 17, 2025

Copy link
Copy Markdown
Member

Fixes #983 .

Changes proposed in this PR:

  • use retry options provided be es
  • update env Readme
  • only use elastic helper for connecting to elastic database

@bogdanfazakas
bogdanfazakas marked this pull request as ready for review July 17, 2025 06:13

@mariacarmina mariacarmina left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@giurgiur99 giurgiur99 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two questions from my side, thanks!

Comment thread src/components/database/ElasticsearchConfigHelper.ts Outdated
Comment thread src/components/database/ElasticsearchConfigHelper.ts Outdated

@giurgiur99 giurgiur99 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@bogdanfazakas
bogdanfazakas merged commit 26c878d into main Jul 18, 2025
13 checks passed
@bogdanfazakas
bogdanfazakas deleted the fix/add-elastic-retry-mech branch July 18, 2025 12:38
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.

Create a retry mechanism in ocean node when connecting to ES

3 participants