Unverified Commit 6420816b authored by John DiSanti's avatar John DiSanti Committed by GitHub
Browse files

Fix the previous release benchmark (#3270) 

The benchmark was broken by the behavior major versions change.

----

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

aws/sdk/benchmarks/previous-release-comparison/Cargo.lock

+394 −313

File changed.

Preview size limit exceeded, changes collapsed.

aws/sdk/benchmarks/previous-release-comparison/Cargo.toml

+2 −2
Original line number Diff line number Diff line
@@ -12,8 +12,8 @@ aws-sdk-s3 = { path = "../../build/aws-sdk/sdk/s3" } 
aws-smithy-runtime = { path = "../../build/aws-sdk/sdk/aws-smithy-runtime", features = ["test-util"] }

criterion = { version = "0.5", features = ["async_tokio"] }

http = "0.2.3"

previous-s3 = { version = "0.35", package = "aws-sdk-s3", features = ["test-util"] }

previous-runtime = { version = "0.57.1", package = "aws-smithy-runtime", features = ["test-util"] }

previous-s3 = { version = "1", package = "aws-sdk-s3", features = ["test-util"] }

previous-runtime = { version = "1", package = "aws-smithy-runtime", features = ["test-util"] }

tokio = { version = "1.23.1", features = ["macros", "test-util", "rt-multi-thread"] }



[profile.release]

aws/sdk/benchmarks/previous-release-comparison/benches/previous_release_comparison.rs

+2 −0
Original line number Diff line number Diff line
@@ -66,6 +66,7 @@ fn bench(c: &mut Criterion) { 
    let main_client = {

        let http_client = test_client!(main);

        let config = aws_sdk_s3::Config::builder()

            .behavior_version(aws_sdk_s3::config::BehaviorVersion::latest())

            .credentials_provider(aws_sdk_s3::config::Credentials::for_tests())

            .region(aws_sdk_s3::config::Region::new("us-east-1"))

            .http_client(http_client)
@@ -75,6 +76,7 @@ fn bench(c: &mut Criterion) { 
    let previous_client = {

        let http_client = test_client!(previous);

        let config = previous_s3::Config::builder()

            .behavior_version(previous_s3::config::BehaviorVersion::latest())

            .credentials_provider(previous_s3::config::Credentials::for_tests())

            .region(previous_s3::config::Region::new("us-east-1"))

            .http_client(http_client)