Skip to content

Commit a93b6db

Browse files
Cosmos: fix pylint min/max-builtin and cspell ppaf in metadata failover grace
Resolves CI Analyze (pylint R1730 consider-using-min-builtin) and cspell (unknown word 'ppaf') failures on PR #47513. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 8bdfc08 commit a93b6db

3 files changed

Lines changed: 4 additions & 4 deletions

File tree

sdk/cosmos/azure-cosmos/azure/cosmos/_metadata_failover_grace.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -50,10 +50,8 @@ def get_grace_seconds() -> float:
5050
value = float(raw)
5151
except (TypeError, ValueError):
5252
value = _Constants.METADATA_FAILOVER_GRACE_SECONDS_DEFAULT
53-
if value < 0:
54-
value = 0.0
55-
if value > _Constants.METADATA_FAILOVER_GRACE_SECONDS_MAX:
56-
value = _Constants.METADATA_FAILOVER_GRACE_SECONDS_MAX
53+
value = max(value, 0.0)
54+
value = min(value, _Constants.METADATA_FAILOVER_GRACE_SECONDS_MAX)
5755
return value
5856

5957

sdk/cosmos/azure-cosmos/tests/test_metadata_failover_grace_unit.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
cross-region failover decision (azure-sdk-for-python#46471 /
99
azure-cosmos-dotnet-v3#5805).
1010
"""
11+
# cspell:ignore ppaf
1112

1213
import asyncio
1314
import time

sdk/cosmos/azure-cosmos/tests/test_metadata_failover_grace_unit_async.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
control-plane metadata (collection) read preempts the cross-region failover
99
decision (azure-sdk-for-python#46471 / azure-cosmos-dotnet-v3#5805).
1010
"""
11+
# cspell:ignore ppaf
1112

1213
import asyncio
1314
import os

0 commit comments

Comments
 (0)