RFC30

[thomas-k-cameron]

Motivation and Context

This PR implements features described on RFC30 except for compile time benchmark.
Benchmark is submitted as a separate PR #2617.

Once this PR and #2617 is merged, RFC30 is complete.

Initially, we tried to make commit to unstable-serde-support branch and merge changes one by one in small PRs. However, in order to make it up to date with the main branch, we would need to go through a large PR of over 700 files.
By merging it directly to the main branch, we can reduce this to under 30 files.

Prerequisite PRs

This is a list of prerequisite PRs; You can reduce the diff if you merge them earlier.
None of them introduces breaking changes.

• Add RUSTFLAGS = aws_sdk_unstable as environment variable when running Cargo from kotlin #2614
• Refactor aws smithy types #2638
• Add serde support to Blob type #2647
• Add serde support to number type #2645
• Add serde support to date time #2646
• Add serde support to document type. #2616
• RFC30: add Serde Decorator to RustClientCodegenPlugin #2650
• Add feature gates and serde related types #2651
• Add send_with method to fluent builders  #2652

EDIT:
More sub-PR.

• RFC30: Add tests to ensure that serde attributes are not added to error types #2803
• Refactoring and adding aws_sdk_unstable to RUSTFLAG on one of the kotlin test #2804

EDIT2:
More sub-PR.

• RFC30: Refactoring blob #2850
• RFC30: Add test that runs with unstable flag on #2851

Description

List of changes:

• rust runtime

  • Implements serde support to Number, Blob, Document, DateTime

• buildSrc

  • add rustflag to enable feature gated features

• Core codegen code

  • Add sensitive warning
  • Add serde attributes
  • Import serde crate to enable serde(skip) on some namespaces
  • Add serde crate behind unstable feature gate on Cargo.toml

• Client codegen code

  • add serde as dependency behind feature gate

Testing

• rust runtime
  • tests whether de/ser produces value as expected

Checklist

• I have updated CHANGELOG.next.toml if I made changes to the smithy-rs codegen or runtime crates
• I have updated CHANGELOG.next.toml if I made changes to the AWS SDK, generated SDK code, or SDK runtime crates

---

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[thomas-k-cameron]

NOTE: this one's fixed.

Only error that I'm facing is this, which comes from tracing crate.
It appears that it's failing to compile regex but I'm not sure where it's coming from.

failures:

---- cache::lazy_caching::tests::load_contention stdout ----
thread 'cache::lazy_caching::tests::load_contention' panicked at 'called `Result::unwrap()` on an `Err` value: Syntax(
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
regex parse error:
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 1: (?x)
 2:             ^(?P<global_level>(?i:trace|debug|info|warn|error|off|[0-5]))$ |
                                      ^
 3:                 #                 ^^^.
 4:                 #                     `note: we match log level names case-insensitively
 5:             ^
 6:             (?: # target name or span name
 7:                 (?P<target>[\w:-]+)|(?P<span>\[[^\]]*\])
 8:             ){1,2}
error: test failed, to rerun pass `--lib`
 9:             (?: # level or nothing
10:                 =(?P<level>(?i:trace|debug|info|warn|error|off|[0-5]))?
11:                     #          ^^^.
12:                     #              `note: we match log level names case-insensitively
13:             )?
14:             $
15:             
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
error: Unicode-aware case insensitivity matching is not available (make sure the unicode-case feature is enabled)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
)', /opt/cargo/registry/src/github.com-1ecc6299db9ec823/tracing-subscriber-0.3.16/src/filter/env/directive.rs:140:10
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

UPDATE: fixed tokio-rs/tracing#2565

[thomas-k-cameron]

@Velfi

So I reorganized the PR/code.

I really wanted to make use of unstable-serde branch but it turned out to be difficult without producing large number of file changes.
I'm very sorry for wasting you and your team's time that was spent on reviewing the PRs for said branch.

I really hopes this approach works.

Thank you very much for you and your team's continuous support!

[thomas-k-cameron]

Never seen this error before.

error: all variants have the same postfix: `Operation`
    --> misc/rust-server-codegen-python/src/service.rs:1326:1
     |
1326 | / pub enum Operation {
1327 | |     RequiredHeaderCollectionOperation,
1328 | |     RequiredInnerShapeOperation,
1329 | |     ResponseCodeDefaultOperation,
...    |
1332 | |     TypeComplexityOperation,
1333 | | }
     | |_^
     |
     = help: remove the postfixes and use full paths to the variants instead of glob imports
     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#enum_variant_names