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

feat: Add support for eventstream input & initial request to RPC-based protocols #1377

Merged
merged 12 commits into from
Mar 15, 2024

Conversation

sichanyoo
Copy link
Contributor

@sichanyoo sichanyoo commented Mar 4, 2024

Issue #

Companion PR: smithy-lang/smithy-swift#675

For RPC-based protocols:

Eventstream input ticket: #1175
Initial request ticket: #1172

Description of changes

  • Add codegen tests & runtime tests for event stream input handling & initial request handling code added in smithy-swift's companion PR.

New/existing dependencies impact assessment, if applicable

Conventional Commits

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@sichanyoo sichanyoo had a problem deploying to Codegen-Build-Test March 4, 2024 22:02 — with GitHub Actions Failure
@sichanyoo sichanyoo temporarily deployed to Integration-Test March 4, 2024 22:02 — with GitHub Actions Inactive
@sichanyoo sichanyoo temporarily deployed to Integration-Test March 4, 2024 22:02 — with GitHub Actions Inactive
@sichanyoo sichanyoo temporarily deployed to Integration-Test March 4, 2024 22:02 — with GitHub Actions Inactive
@sichanyoo sichanyoo temporarily deployed to Integration-Test March 4, 2024 22:03 — with GitHub Actions Inactive
@sichanyoo sichanyoo requested a review from jbelkins March 4, 2024 22:07
Sichan Yoo added 4 commits March 7, 2024 16:31
…hapes in extension when: 1) protocol uses RPC architecture 2) protocol supports event streaming 3) input shape at hand has streaming union member. The generated method returns EventStream.Message from the input struct, with non-sreaming members encoded into payload & :event-type: initial-request header in its headers.
…n for generating encodable conformances, like restJson1 does.
…/ add codegen tests for event stream input and iniital request for awsJson protocol codegen.
@sichanyoo sichanyoo had a problem deploying to Codegen-Build-Test March 8, 2024 00:56 — with GitHub Actions Failure
@sichanyoo sichanyoo temporarily deployed to Integration-Test March 8, 2024 00:56 — with GitHub Actions Inactive
@sichanyoo sichanyoo temporarily deployed to Integration-Test March 8, 2024 00:56 — with GitHub Actions Inactive
@sichanyoo sichanyoo temporarily deployed to Integration-Test March 8, 2024 00:56 — with GitHub Actions Inactive
@sichanyoo sichanyoo temporarily deployed to Integration-Test March 8, 2024 00:56 — with GitHub Actions Inactive
@sichanyoo sichanyoo temporarily deployed to Integration-Test March 8, 2024 01:23 — with GitHub Actions Inactive
@sichanyoo sichanyoo temporarily deployed to Codegen-Build-Test March 8, 2024 01:23 — with GitHub Actions Inactive
@sichanyoo sichanyoo temporarily deployed to Integration-Test March 8, 2024 01:23 — with GitHub Actions Inactive
@sichanyoo sichanyoo temporarily deployed to Integration-Test March 8, 2024 01:23 — with GitHub Actions Inactive
@sichanyoo sichanyoo temporarily deployed to Integration-Test March 8, 2024 01:23 — with GitHub Actions Inactive
Sichan Yoo added 2 commits March 13, 2024 14:44
…quest method to smithy-swift. Rename package in integrations list to use the new package location.
@sichanyoo sichanyoo temporarily deployed to Integration-Test March 13, 2024 21:45 — with GitHub Actions Inactive
@sichanyoo sichanyoo had a problem deploying to Codegen-Build-Test March 13, 2024 21:45 — with GitHub Actions Failure
@sichanyoo sichanyoo temporarily deployed to Integration-Test March 13, 2024 21:45 — with GitHub Actions Inactive
@sichanyoo sichanyoo temporarily deployed to Integration-Test March 13, 2024 21:45 — with GitHub Actions Inactive
@sichanyoo sichanyoo temporarily deployed to Integration-Test March 13, 2024 21:45 — with GitHub Actions Inactive
@sichanyoo sichanyoo merged commit 457216e into main Mar 15, 2024
17 checks passed
@sichanyoo sichanyoo deleted the feat/eventstream-input-awsJson branch March 15, 2024 18:04
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.

2 participants