Bump JDBC driver's dependency version by dai-chen · Pull Request #735 · opensearch-project/sql · GitHub
Skip to content

Bump JDBC driver's dependency version#735

Merged
dai-chen merged 4 commits into
opensearch-project:mainfrom
dai-chen:bump-dependency-version-new
Aug 9, 2022
Merged

Bump JDBC driver's dependency version#735
dai-chen merged 4 commits into
opensearch-project:mainfrom
dai-chen:bump-dependency-version-new

Conversation

@dai-chen

@dai-chen dai-chen commented Aug 8, 2022

Copy link
Copy Markdown
Collaborator

Description

Bump dependency version:

  1. Httpclient
  2. Wiremock (test dependency)
  3. Guava (test dependency introduced by wiremock)
  4. Json-smart (test dependency introduced by wiremock)

Issues Resolved

N/A

Check List

  • New functionality includes testing.
    • All tests pass, including unit test, integration test and doctest
  • New functionality has been documented.
    • New functionality has javadoc added
    • New functionality has user manual doc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Chen Dai <daichen@amazon.com>
Signed-off-by: Chen Dai <daichen@amazon.com>
@dai-chen dai-chen added the JDBC label Aug 8, 2022
@dai-chen dai-chen self-assigned this Aug 8, 2022
@dai-chen dai-chen marked this pull request as ready for review August 8, 2022 23:30
@dai-chen dai-chen requested a review from a team as a code owner August 8, 2022 23:30
@codecov-commenter

Copy link
Copy Markdown

@dai-chen dai-chen merged commit 29f1c52 into opensearch-project:main Aug 9, 2022
@dai-chen dai-chen deleted the bump-dependency-version-new branch August 9, 2022 18:04
@dai-chen dai-chen added the dependencies Pull requests that update a dependency file label Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants