Azure / azure-cosmos-dotnet-v3

.NET SDK for Azure Cosmos DB for the core SQL API
MIT License
736 stars 491 forks source link

FaultInjection: Refactors applyPercentage name to injectionRate to make feature clearer. #4511

Closed NaluTripician closed 4 months ago

NaluTripician commented 4 months ago

Pull Request Template

Description

Changed the name of the ApplyPercentage feature to InjectionRate. This should make the function of the feature clearer to users. This will also be the name of the feature in the Java SDK.

Type of change

Please delete options that are not relevant.

Closing issues

To automatically close an issue: closes #IssueNumber

NaluTripician commented 4 months ago

LGTM. Let's verify why the tests are failing.

Looks like they are from some flaky tests. They seem to be failing on other PRs too and I have seen these randomly fail before. I will rerun the test a few more time to double check but currently there are no tests that rely on this.