Skip to content

Update OpenSearch Java Client documentation: update version, add ApacheHttpClient5Transport, fix code samples#3510

Merged
Naarcha-AWS merged 2 commits intoopensearch-project:mainfrom
reta:issue-409
Mar 21, 2023
Merged

Update OpenSearch Java Client documentation: update version, add ApacheHttpClient5Transport, fix code samples#3510
Naarcha-AWS merged 2 commits intoopensearch-project:mainfrom
reta:issue-409

Conversation

@reta
Copy link
Contributor

@reta reta commented Mar 16, 2023

Description

Update OpenSearch Java Client documentation: update version, add ApacheHttpClient5Transport, fix code samples

Issues Resolved

Closes opensearch-project/opensearch-java#409

Checklist

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

…heHttpClient5Transport, fix code samples

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
@reta reta requested a review from a team as a code owner March 16, 2023 20:37
@reta
Copy link
Contributor Author

reta commented Mar 16, 2023

@VachaShah @hdhalter @dblock please take a look, thank you!

Copy link
Collaborator

@kolchfa-aws kolchfa-aws left a comment

Choose a reason for hiding this comment

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

LGTM. Made a couple of suggestions. Thanks, @reta!

This getting started guide illustrates how to connect to OpenSearch, index documents, and run queries. For the client source code, see the [opensearch-java repo](https://github.com/opensearch-project/opensearch-java).

## Installing the client
## Installing the client using Apache HttpClient 5 Transport
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
## Installing the client using Apache HttpClient 5 Transport
## Installing the client using Apache HttpClient 5 Transport

_clients/java.md Outdated

## Installing the client
## Installing the client using Apache HttpClient 5 Transport
To start using the OpenSearch Java client, you need to provide the transport. The default one is `ApacheHttpClient5TransportBuilder` and comes with the Java client.
Copy link
Collaborator

@kolchfa-aws kolchfa-aws Mar 17, 2023

Choose a reason for hiding this comment

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

Suggested change
To start using the OpenSearch Java client, you need to provide the transport. The default one is `ApacheHttpClient5TransportBuilder` and comes with the Java client.
To start using the OpenSearch Java client, you need to provide a transport. The default `ApacheHttpClient5TransportBuilder` transport comes with the Java client. To use the OpenSearch Java client with the default transport, add it to your `pom.xml` file as a dependency:

_clients/java.md Outdated
```
{% include copy.html %}

If you're using Gradle, add the following dependencies to your project.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
If you're using Gradle, add the following dependencies to your project.
If you're using Gradle, add the following dependencies to your project:

_clients/java.md Outdated

## Installing the client using RestClient Transport

The alternative way to create a Java client is by using `RestClient` based transport, ensure that you have the following dependencies in your project's `pom.xml` file:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
The alternative way to create a Java client is by using `RestClient` based transport, ensure that you have the following dependencies in your project's `pom.xml` file:
Alternatively, you can create a Java client by using the `RestClient`-based transport. In this case, make sure that you have the following dependencies in your project's `pom.xml` file:

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
@reta
Copy link
Contributor Author

reta commented Mar 17, 2023

LGTM. Made a couple of suggestions. Thanks, @reta!

Thanks @kolchfa-aws , comments addressed!

@Naarcha-AWS Naarcha-AWS added backport 1.3 PR: Backport label for v1.3.x backport 2.5 PR: Backport label for 2.5 backport 2.6 PR: Backport label for 2.6 labels Mar 21, 2023
@Naarcha-AWS Naarcha-AWS merged commit 4e58f94 into opensearch-project:main Mar 21, 2023
opensearch-trigger-bot bot pushed a commit that referenced this pull request Mar 21, 2023
…heHttpClient5Transport, fix code samples (#3510)

* Update OpenSearch Java Client documentation: update version, add ApacheHttpClient5Transport, fix code samples

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

* Address review comments

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

---------

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
(cherry picked from commit 4e58f94)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Mar 21, 2023
…heHttpClient5Transport, fix code samples (#3510)

* Update OpenSearch Java Client documentation: update version, add ApacheHttpClient5Transport, fix code samples

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

* Address review comments

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

---------

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
(cherry picked from commit 4e58f94)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Mar 21, 2023
…heHttpClient5Transport, fix code samples (#3510)

* Update OpenSearch Java Client documentation: update version, add ApacheHttpClient5Transport, fix code samples

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

* Address review comments

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

---------

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
(cherry picked from commit 4e58f94)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Naarcha-AWS pushed a commit that referenced this pull request Mar 21, 2023
…heHttpClient5Transport, fix code samples (#3510) (#3537)

* Update OpenSearch Java Client documentation: update version, add ApacheHttpClient5Transport, fix code samples



* Address review comments



---------


(cherry picked from commit 4e58f94)

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Naarcha-AWS pushed a commit that referenced this pull request Mar 21, 2023
…heHttpClient5Transport, fix code samples (#3510) (#3536)

* Update OpenSearch Java Client documentation: update version, add ApacheHttpClient5Transport, fix code samples



* Address review comments



---------


(cherry picked from commit 4e58f94)

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Naarcha-AWS pushed a commit that referenced this pull request Mar 21, 2023
…heHttpClient5Transport, fix code samples (#3510) (#3535)

* Update OpenSearch Java Client documentation: update version, add ApacheHttpClient5Transport, fix code samples



* Address review comments



---------


(cherry picked from commit 4e58f94)

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
vagimeli pushed a commit that referenced this pull request May 4, 2023
…heHttpClient5Transport, fix code samples (#3510)

* Update OpenSearch Java Client documentation: update version, add ApacheHttpClient5Transport, fix code samples

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

* Address review comments

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

---------

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
harshavamsi pushed a commit to harshavamsi/documentation-website that referenced this pull request Oct 31, 2023
…heHttpClient5Transport, fix code samples (opensearch-project#3510)

* Update OpenSearch Java Client documentation: update version, add ApacheHttpClient5Transport, fix code samples

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

* Address review comments

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

---------

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport 1.3 PR: Backport label for v1.3.x backport 2.5 PR: Backport label for 2.5 backport 2.6 PR: Backport label for 2.6

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Feedback on Java Client documentation

3 participants