Skip to content

Commit e26dcdd

Browse files
simorenohCopilot
andcommitted
Fix post-merge clippy errors in retry_evaluation
Remove dead cosmos_headers and body params from try_handle_retry_trigger_group — only needed by the !safe_to_retry abort path removed in the idempotency gate removal. Remove stale status field from Abort variant after CosmosError refactor (#4442). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent af11d2b commit e26dcdd

1 file changed

Lines changed: 3 additions & 12 deletions

File tree

sdk/cosmos/azure_data_cosmos_driver/src/driver/pipeline/retry_evaluation.rs

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -252,15 +252,9 @@ fn evaluate_http_outcome(
252252
return result;
253253
}
254254

255-
if let Some(result) = try_handle_retry_trigger_group(
256-
operation,
257-
endpoint,
258-
retry_state,
259-
&status,
260-
&cosmos_headers,
261-
&body,
262-
request_sent,
263-
) {
255+
if let Some(result) =
256+
try_handle_retry_trigger_group(operation, endpoint, retry_state, &status, request_sent)
257+
{
264258
return result;
265259
}
266260

@@ -401,8 +395,6 @@ fn try_handle_retry_trigger_group(
401395
endpoint: &CosmosEndpoint,
402396
retry_state: &OperationRetryState,
403397
status: &CosmosStatus,
404-
cosmos_headers: &CosmosResponseHeaders,
405-
body: &[u8],
406398
request_sent: RequestSentStatus,
407399
) -> Option<(OperationAction, Vec<LocationEffect>)> {
408400
let is_system_resource_unavailable = status.is_throttled()
@@ -547,7 +539,6 @@ fn evaluate_transport_layer_outcome(
547539
return (
548540
OperationAction::Abort {
549541
error: build_transport_error(&status, error),
550-
status: Some(status),
551542
},
552543
effects,
553544
);

0 commit comments

Comments
 (0)