Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

aws crt client #64

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

aws crt client #64

wants to merge 1 commit into from

Conversation

sullis
Copy link
Contributor

@sullis sullis commented Jul 25, 2024

This is a Draft PR.

The purpose of this PR is to evaluate Amazon's CRT client

CRT client availability

The AWS CRT client was announced in February 2023
https://aws.amazon.com/blogs/developer/announcing-availability-of-the-aws-crt-http-client-in-the-aws-sdk-for-java-2-x/

Reference

https://docs.aws.amazon.com/sdk-for-java/latest/developer-guide/http-configuration.html

https://docs.aws.amazon.com/sdk-for-java/latest/developer-guide/http-configuration-crt.html

Amazon's flowchart

According to Amazon's flowchart, the CRT client is preferred for most applications.

@sullis
Copy link
Contributor Author

sullis commented Aug 8, 2024

I have been evaluating the AWS CRT Client in a different repo:

https://github.com/sullis/s3-playground/tree/main/src/test/java/io/github/sullis/s3/playground

private static final List<SdkAsyncHttpClient.Builder<?>> ASYNC_HTTP_CLIENT_BUILDER_LIST =
      List.of(NettyNioAsyncHttpClient.builder(), AwsCrtAsyncHttpClient.builder());

The s3-playground project is experimental code :)

@sullis sullis marked this pull request as ready for review September 19, 2024 22:51
@sullis sullis marked this pull request as draft September 19, 2024 22:52
.maxConcurrency(clientConfig.maxConcurrency)
clientConfig.readWriteTimeout?.also {
builder.readTimeout(it).writeTimeout(it)
// TODO fixme
Copy link
Owner

Choose a reason for hiding this comment

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

does the CRT client just not have this feature?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants