From ca700b01485cc18d744bda1f227a6f3e94a2d72d Mon Sep 17 00:00:00 2001 From: Thomas Cameron <68596478+thomas-k-cameron@users.noreply.github.com> Date: Fri, 13 Jan 2023 21:46:14 +0000 Subject: [PATCH] [smithy-rs] RFC: Serialization and deserialization (#1944) * commit RFC * commit * Delete rfc0023_serialization_and_deserialization.md * fix formatting * https://github.com/awslabs/smithy-rs/pull/1944#discussion_r1054637168 * https://github.com/awslabs/smithy-rs/pull/1944#discussion_r1054632202 * applied grammarly pre-commit * better example wip * FIX * https://github.com/awslabs/smithy-rs/pull/1944#discussion_r1054632468 https://github.com/awslabs/smithy-rs/pull/1944#discussion_r1054818444 https://github.com/awslabs/smithy-rs/pull/1944#discussion_r1054820192 * add examples for output's builer * fixes * fixing unstable feature gate snippet * Update rfc0028_serialization_and_deserialization.md Section is deleted in response to the discussion https://github.com/awslabs/smithy-rs/pull/1944#discussion_r1061577656 * file name fix --- sdk/kinesis/src/input.rs | 2 +- sdk/s3control/src/input.rs | 22 +++++++++++----------- versions.toml | 6 +++--- 3 files changed, 15 insertions(+), 15 deletions(-) diff --git a/sdk/kinesis/src/input.rs b/sdk/kinesis/src/input.rs index 8ff704a33246..dd4965c1b723 100644 --- a/sdk/kinesis/src/input.rs +++ b/sdk/kinesis/src/input.rs @@ -1396,8 +1396,8 @@ impl DescribeStreamConsumerInput { .set_use_fips(_config.use_fips) .set_use_dual_stack(_config.use_dual_stack) .set_operation_type(Some("control".to_string())) - .set_stream_arn(self.stream_arn.clone()) .set_consumer_arn(self.consumer_arn.clone()) + .set_stream_arn(self.stream_arn.clone()) .build() .map_err(|err| { aws_smithy_http::endpoint::ResolveEndpointError::from_source( diff --git a/sdk/s3control/src/input.rs b/sdk/s3control/src/input.rs index 405c311083c6..7a92862bba0c 100644 --- a/sdk/s3control/src/input.rs +++ b/sdk/s3control/src/input.rs @@ -1473,8 +1473,8 @@ impl DeleteAccessPointInput { .set_use_fips(_config.use_fips) .set_use_dual_stack(_config.use_dual_stack) .set_requires_account_id(Some(true)) - .set_access_point_name(self.name.clone()) .set_account_id(self.account_id.clone()) + .set_access_point_name(self.name.clone()) .build() .map_err(|err| { aws_smithy_http::endpoint::ResolveEndpointError::from_source( @@ -2470,8 +2470,8 @@ impl DeleteBucketLifecycleConfigurationInput { .set_use_fips(_config.use_fips) .set_use_dual_stack(_config.use_dual_stack) .set_requires_account_id(Some(true)) - .set_bucket(self.bucket.clone()) .set_account_id(self.account_id.clone()) + .set_bucket(self.bucket.clone()) .build() .map_err(|err| { aws_smithy_http::endpoint::ResolveEndpointError::from_source( @@ -2683,8 +2683,8 @@ impl DeleteBucketPolicyInput { .set_use_fips(_config.use_fips) .set_use_dual_stack(_config.use_dual_stack) .set_requires_account_id(Some(true)) - .set_account_id(self.account_id.clone()) .set_bucket(self.bucket.clone()) + .set_account_id(self.account_id.clone()) .build() .map_err(|err| { aws_smithy_http::endpoint::ResolveEndpointError::from_source( @@ -2890,8 +2890,8 @@ impl DeleteBucketTaggingInput { .set_use_fips(_config.use_fips) .set_use_dual_stack(_config.use_dual_stack) .set_requires_account_id(Some(true)) - .set_account_id(self.account_id.clone()) .set_bucket(self.bucket.clone()) + .set_account_id(self.account_id.clone()) .build() .map_err(|err| { aws_smithy_http::endpoint::ResolveEndpointError::from_source( @@ -4990,8 +4990,8 @@ impl GetAccessPointPolicyInput { .set_use_fips(_config.use_fips) .set_use_dual_stack(_config.use_dual_stack) .set_requires_account_id(Some(true)) - .set_account_id(self.account_id.clone()) .set_access_point_name(self.name.clone()) + .set_account_id(self.account_id.clone()) .build() .map_err(|err| { aws_smithy_http::endpoint::ResolveEndpointError::from_source( @@ -5762,8 +5762,8 @@ impl GetBucketInput { .set_use_fips(_config.use_fips) .set_use_dual_stack(_config.use_dual_stack) .set_requires_account_id(Some(true)) - .set_bucket(self.bucket.clone()) .set_account_id(self.account_id.clone()) + .set_bucket(self.bucket.clone()) .build() .map_err(|err| { aws_smithy_http::endpoint::ResolveEndpointError::from_source( @@ -5967,8 +5967,8 @@ impl GetBucketLifecycleConfigurationInput { .set_use_fips(_config.use_fips) .set_use_dual_stack(_config.use_dual_stack) .set_requires_account_id(Some(true)) - .set_bucket(self.bucket.clone()) .set_account_id(self.account_id.clone()) + .set_bucket(self.bucket.clone()) .build() .map_err(|err| { aws_smithy_http::endpoint::ResolveEndpointError::from_source( @@ -6178,8 +6178,8 @@ impl GetBucketPolicyInput { .set_use_fips(_config.use_fips) .set_use_dual_stack(_config.use_dual_stack) .set_requires_account_id(Some(true)) - .set_bucket(self.bucket.clone()) .set_account_id(self.account_id.clone()) + .set_bucket(self.bucket.clone()) .build() .map_err(|err| { aws_smithy_http::endpoint::ResolveEndpointError::from_source( @@ -8316,8 +8316,8 @@ impl ListAccessPointsInput { .set_use_fips(_config.use_fips) .set_use_dual_stack(_config.use_dual_stack) .set_requires_account_id(Some(true)) - .set_bucket(self.bucket.clone()) .set_account_id(self.account_id.clone()) + .set_bucket(self.bucket.clone()) .build() .map_err(|err| { aws_smithy_http::endpoint::ResolveEndpointError::from_source( @@ -10476,8 +10476,8 @@ impl PutBucketPolicyInput { .set_use_fips(_config.use_fips) .set_use_dual_stack(_config.use_dual_stack) .set_requires_account_id(Some(true)) - .set_account_id(self.account_id.clone()) .set_bucket(self.bucket.clone()) + .set_account_id(self.account_id.clone()) .build() .map_err(|err| { aws_smithy_http::endpoint::ResolveEndpointError::from_source( @@ -10723,8 +10723,8 @@ impl PutBucketTaggingInput { .set_use_fips(_config.use_fips) .set_use_dual_stack(_config.use_dual_stack) .set_requires_account_id(Some(true)) - .set_account_id(self.account_id.clone()) .set_bucket(self.bucket.clone()) + .set_account_id(self.account_id.clone()) .build() .map_err(|err| { aws_smithy_http::endpoint::ResolveEndpointError::from_source( diff --git a/versions.toml b/versions.toml index 7fd7577d4099..16861569893a 100644 --- a/versions.toml +++ b/versions.toml @@ -1,4 +1,4 @@ -smithy_rs_revision = '0ab5493762ab39d44043f8122774fb8825605e55' +smithy_rs_revision = '59f7e950473cb87beddf20ef878547adbb0dfa94' aws_doc_sdk_examples_revision = 'cf400c3d1eefb910869a961bc3c948e75386daa1' [manual_interventions] @@ -1051,7 +1051,7 @@ model_hash = '6556c6b5343776c5dea4f1bdb3f5bd03ba10f147b6b74218f8b6d54efdae74ca' [crates.aws-sdk-kinesis] category = 'AwsSdk' version = '0.23.0' -source_hash = '6e23a6f8e1ab1fcea56afa4aadb5ab1c04fb8c0e7f708729749d642165b5f951' +source_hash = '614b335f12573e08dc73448177e1651134159af46be2f2549d8946342737e2f5' model_hash = 'd3fe0055424ca8c9d4998ffb25623977d28d2d5bebd852805c17787809c745f2' [crates.aws-sdk-kinesisanalytics] @@ -1663,7 +1663,7 @@ model_hash = '78eb7dd7acf02fc39b4ec540a7135fb1846a13888ccc292d1dc105b80b696757' [crates.aws-sdk-s3control] category = 'AwsSdk' version = '0.23.0' -source_hash = 'e4da460b22ac19796617679f453ca705f3c4a61def5ec5b83121a88222f68aa8' +source_hash = 'fb53e3d979b81fc6954ee41d980f1cf421f74fd6822d5450008853f6f4e145d5' model_hash = 'bd3252c3f5ee09b9431d77bd3dd985679bd3b94d7b61cb90727c63cbd0653833' [crates.aws-sdk-s3outposts]