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

[improve] Upgrade opensearch sink to client 2.16 and tests to use server 2.16.0 #313

Merged
merged 18 commits into from
Sep 19, 2024

Conversation

dlg99
Copy link
Collaborator

@dlg99 dlg99 commented Sep 10, 2024

Motivation

Upgrade opensearch sink to client 2.16 and tests to use server 2.16.0

Modifications

Upgraded opensearch sink to client 2.16 and updated tests to use server 2.16.0

Verifying this change

  • Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.

Does this pull request potentially affect one of the following parts:

NO

If yes was chosen, please highlight the changes

  • Dependencies (does it add or upgrade a dependency): (yes / no)
  • The public API: (yes / no)
  • The schema: (yes / no / don't know)
  • The default values of configurations: (yes / no)
  • The wire protocol: (yes / no)
  • The rest endpoints: (yes / no)
  • The admin cli options: (yes / no)
  • Anything that affects deployment: (yes / no / don't know)

Documentation

Check the box below or label this PR directly (if you have committer privilege).

Need to update docs?

  • doc-required

    (If you need help on updating docs, create a doc issue)

  • no-need-doc

    (Please explain why)

  • doc

    (If this PR contains doc changes)

@dlg99 dlg99 closed this Sep 12, 2024
@dlg99 dlg99 reopened this Sep 12, 2024
@dlg99 dlg99 changed the title test with opensearch server 2.16.0 [improve] Upgrade opensearch sink to client 2.16 and tests to use server 2.16.0 Sep 14, 2024
@srinath-ctds srinath-ctds merged commit 1dfac82 into datastax:2.10_ds Sep 19, 2024
19 of 27 checks passed
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.

4 participants