Skip to content

[Thin Client Integration] Benchmark tests: Adds tests for thinclient mode dataplane scenarios.#5228

Merged
kirankumarkolli merged 21 commits intomasterfrom
users/aavasthy/5166_AddBenchmarkTests
Jun 27, 2025
Merged

[Thin Client Integration] Benchmark tests: Adds tests for thinclient mode dataplane scenarios.#5228
kirankumarkolli merged 21 commits intomasterfrom
users/aavasthy/5166_AddBenchmarkTests

Conversation

@aavasthy
Copy link
Copy Markdown
Contributor

@aavasthy aavasthy commented Jun 2, 2025

Pull Request Template

Description

Add benchmark tests for Azure Cosmos DB in ThinClient mode.

Testing the thinclient benchmark tests:

  • When running benchmark on an azure VM then run summary for these tests get added to the benchmark account containers.
image
  • When thinclient is enabled, tests use Gateway mode and set thinclient environment variable to true.
image - image image
  • Also regression tested this change for when thinclient flag is disabled. It then uses the other account and runs the existing workload.

Data Analysis workbook: https://adb-7071740625730289.9.azuredatabricks.net/editor/notebooks/4229795170788257?o=7071740625730289#command/6742845551923668
This change runs in master against a prod account and thinclient is currently not deployed in prod. The above analysis workbook is running against a private VM and stage account.
image

Type of change

  • New feature (non-breaking change which adds functionality)

Closing issues

To automatically close an issue: closes #5166

@aavasthy aavasthy marked this pull request as ready for review June 3, 2025 14:54
@aavasthy aavasthy force-pushed the users/aavasthy/5166_AddBenchmarkTests branch from 351b355 to 7466146 Compare June 4, 2025 23:51
Comment thread Microsoft.Azure.Cosmos.Samples/Tools/Benchmark/BenchmarkConfig.cs Outdated
Comment thread Microsoft.Azure.Cosmos.Samples/Tools/Benchmark/loop.sh
Comment thread Microsoft.Azure.Cosmos.Samples/Tools/Benchmark/run.sh
Comment thread Microsoft.Azure.Cosmos.Samples/Tools/Benchmark/run.sh
Copy link
Copy Markdown
Member

@kundadebdatta kundadebdatta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Member

@FabianMeiswinkel FabianMeiswinkel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM Thanks

@kundadebdatta kundadebdatta added thin-client-integration auto-merge Enables automation to merge PRs and removed auto-merge Enables automation to merge PRs labels Jun 26, 2025
@kirankumarkolli kirankumarkolli merged commit fd0cc4a into master Jun 27, 2025
31 checks passed
@kirankumarkolli kirankumarkolli deleted the users/aavasthy/5166_AddBenchmarkTests branch June 27, 2025 04:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge Enables automation to merge PRs thin-client-integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Thin Client Integration] Add benchmark tests for thinclient mode.

5 participants