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

Add support for sync client APIs to CRT Client #3343

Closed
2 tasks
sebastian-zero opened this issue Aug 4, 2022 · 2 comments
Closed
2 tasks

Add support for sync client APIs to CRT Client #3343

sebastian-zero opened this issue Aug 4, 2022 · 2 comments
Labels
crt-client feature-request A feature should be added or improved. p3 This is a minor priority issue

Comments

@sebastian-zero
Copy link

sebastian-zero commented Aug 4, 2022

Describe the feature

Current CRT client only supports async clients and that requires to rewrite existing sync logic. While it's often just adding .join(), other parts are more complicated to rewrite, e.g. paging logic. Sync client APIs are often much easier to use and still cover most use cases, especially for lambdas.

Use Case

Use cases:

  1. make migration of workloads using sync clients APIs to CRT easier, avoiding rewriting sync logic
  2. make use of easier sync client APIs when implementing new projects against CRT

Proposed Solution

Add sync client APIs to to CRT client.

Other Information

Also see discussion here: #2939

Acknowledgements

  • I may be able to implement this feature request
  • This feature might incur a breaking change

AWS Java SDK version used

latest

JDK version used

all currently supported

Operating System and version

all currently supported

@sebastian-zero sebastian-zero added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Aug 4, 2022
@debora-ito debora-ito changed the title Add support for sync client APIs Add support for sync client APIs to CRT Client Aug 4, 2022
@debora-ito debora-ito added crt-client and removed needs-triage This issue or PR still needs to be triaged. labels Aug 4, 2022
@debora-ito debora-ito added the p3 This is a minor priority issue label Feb 21, 2023
@zoewangg
Copy link
Contributor

AWS CRT sync HTTP client, AwsCrtHttpClient, has been released as part of 2.22.0. It's in the same CRT module as the CRT async HTTP client, aws-crt-client. Closing the issue

Copy link

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
crt-client feature-request A feature should be added or improved. p3 This is a minor priority issue
Projects
None yet
Development

No branches or pull requests

3 participants