[DataProtection] Fix C# backup job duration format - #46623
Minghao Chen (HarveyLink) wants to merge 2 commits into
Conversation
Next Steps to MergeNext steps that must be taken to merge this PR:
Comment generated by summarize-checks workflow run. |
API Change CheckAPIView identified API level changes in this PR and created the following API reviews
Comment generated by After APIView workflow run. |
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
a37340d to
60b0ee9
Compare
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
TypeSpec suppressions requiring review (testing, non-blocking)Status: ❌ Approval required (currently under testing, review NOT enforced) — 1 suppression
New suppressions (1)
For an overview of TypeSpec linting rules click here. 💬 Have feedback on the TypeSpec suppression flow? Let us know. |
Summary
Fixes Azure/azure-sdk-for-net#59573.
The REST contract remains unchanged:
AzureBackupJob.durationis still a string and generated Swagger is unchanged. This client-only customization defines aDurationConstantscalar withduration-constantencoding and applies it only to the C# alternate type forAzureBackupJob.duration. A targetedknown-encodingsuppression documents why this service-specific encoding is required. The C# generator therefore emits constant-format ("c")TimeSpanparsing without changing other language SDKs.Companion SDK draft PR: Azure/azure-sdk-for-net#63305