Skip to content

Comments

[Bundle] Include hardDelete and purge as part of TransactionScope#5366

Open
fhibf wants to merge 3 commits intomainfrom
user/fernfe/bundleSupportHardDelete
Open

[Bundle] Include hardDelete and purge as part of TransactionScope#5366
fhibf wants to merge 3 commits intomainfrom
user/fernfe/bundleSupportHardDelete

Conversation

@fhibf
Copy link
Contributor

@fhibf fhibf commented Feb 3, 2026

Description

In this PR I'm adding Hard Delete and Purge history operations on the same scope of the C# Transactions executed by sequential bundles.

For parallel bundles, as these operations are not using the same stored procedure and C# Transactions, then more work needs to be done. I ended up adding back the validations to block Hard Deletes and Purge History from parallel bundles.

Related issues

Addresses AB#182156

Testing

Describe how this change was tested.

FHIR Team Checklist

  • Update the title of the PR to be succinct and less than 65 characters
  • Add a milestone to the PR for the sprint that it is merged (i.e. add S47)
  • Tag the PR with the type of update: Bug, Build, Dependencies, Enhancement, New-Feature or Documentation
  • Tag the PR with Open source, Azure API for FHIR (CosmosDB or common code) or Azure Healthcare APIs (SQL or common code) to specify where this change is intended to be released.
  • Tag the PR with Schema Version backward compatible or Schema Version backward incompatible or Schema Version unchanged if this adds or updates Sql script which is/is not backward compatible with the code.
  • When changing or adding behavior, if your code modifies the system design or changes design assumptions, please create and include an ADR.
  • CI is green before merge Build Status
  • Review squash-merge requirements

Semver Change (docs)

Patch|Skip|Feature|Breaking (reason)

@fhibf fhibf requested a review from a team as a code owner February 3, 2026 18:31
@fhibf fhibf added Bug Bug bug bug. Azure Healthcare APIs Label denotes that the issue or PR is relevant to the FHIR service in the Azure Healthcare APIs No-PaaS-breaking-change No-ADR ADR not needed labels Feb 3, 2026
@fhibf fhibf added this to the FY26\Q3\2Wk\2Wk16 milestone Feb 3, 2026
@fhibf fhibf changed the title [Bundle] Include hardDelete as part of TransactionScope [Bundle] Include hardDelete and purge as part of TransactionScope Feb 3, 2026
@fhibf fhibf enabled auto-merge (squash) February 4, 2026 15:47
@fhibf
Copy link
Contributor Author

fhibf commented Feb 4, 2026

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@fhibf fhibf disabled auto-merge February 4, 2026 19:53
Copy link
Contributor Author

@fhibf fhibf left a comment

Choose a reason for hiding this comment

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

Waiting for confirmation before merging these changes.

@fhibf
Copy link
Contributor Author

fhibf commented Feb 5, 2026

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Azure Healthcare APIs Label denotes that the issue or PR is relevant to the FHIR service in the Azure Healthcare APIs Bug Bug bug bug. No-ADR ADR not needed No-PaaS-breaking-change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants