-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Smithy refactor Json protocol with Single direction EventStreaming (v4/v4a/bearer) #3212
Merged
Merged
Changes from 2 commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
3973d48
version 0: refactor with smithy client for json protocol non customized
sbera87 8867b8b
cleanup
sbera87 f333753
refactor for smithy/legacy json clients
sbera87 cbc6b8f
tweak
sbera87 a99e702
al2 compilaiton fix, pr comment address
sbera87 7e17cbe
fix for category of services with stream members with examples
sbera87 7b08305
add support for path segment of different types
sbera87 aeb4287
dry run fix for path segments greater than one case
sbera87 a9768c8
exclude customization services and rename method to avoid shadowing c…
sbera87 13393cb
removing streaming
sbera87 790a441
remove event streaming
sbera87 1d7c618
remove bidirectional event streaming changes
sbera87 dec045a
fix to accomodate more clients
sbera87 453f112
fix in codegen for loop variable
sbera87 004145e
change tests for smithy client not copyable
sbera87 b796422
namespace resolution of AWS Error
sbera87 802556a
efs client changes
sbera87 22c5b10
merge changes
sbera87 9c042fa
fix rfc paths for smithy clients
sbera87 601d6e3
fix query string
sbera87 24104d4
changes for presigner
sbera87 19b7945
presigner support
sbera87 65e9c06
merge changes from mainline
sbera87 95b2966
merge useragent changes
sbera87 8e9e896
merges from main
sbera87 a84fdba
fix merge conflict
sbera87 3e3f242
exclude bidirectional streaming client
sbera87 6c73fee
address pr comments + cleanup
sbera87 28f5343
move code to initializer
sbera87 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
1,273 changes: 699 additions & 574 deletions
1,273
generated/src/aws-cpp-sdk-codecatalyst/source/CodeCatalystClient.cpp
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.