Skip to content

[Backport 2.x] Refactor two-client usage.#386

Merged
VachaShah merged 1 commit into2.xfrom
backport/backport-333-to-2.x
Mar 2, 2023
Merged

[Backport 2.x] Refactor two-client usage.#386
VachaShah merged 1 commit into2.xfrom
backport/backport-333-to-2.x

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

Backport a0ca15c from #333.

* Refactor two-client usage.

Signed-off-by: dblock <dblock@amazon.com>

* Avoid double-closing the client on cleanup.

Signed-off-by: dblock <dblock@amazon.com>

* Refactor constructors to take sync/async explicitly.

Signed-off-by: dblock <dblock@amazon.com>

* Corrected comments and cleaned up variable naming.

Signed-off-by: dblock <dblock@amazon.com>

Signed-off-by: dblock <dblock@amazon.com>
(cherry picked from commit a0ca15c)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@VachaShah VachaShah merged commit 63693ab into 2.x Mar 2, 2023
@gaiksaya gaiksaya deleted the backport/backport-333-to-2.x branch May 30, 2024 22:40
@BrendonFaleiro BrendonFaleiro mentioned this pull request Jun 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants