Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

cleanup the retry logic #3429

Merged
merged 1 commit into from
Jul 31, 2023
Merged

cleanup the retry logic #3429

merged 1 commit into from
Jul 31, 2023

Conversation

v-mserdar
Copy link
Contributor

@v-mserdar v-mserdar commented Jul 25, 2023 •
edited
Loading

Description

Fix the retry logic so that it correctly applies the retry logic

Related issues

Addresses [issue #].

Testing

Ran this against my local reindex job

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.
  • CI is green before merge Build Status
  • Review squash-merge requirements

Semver Change (docs)

Patch|Skip|Feature|Breaking (reason)

@v-mserdar v-mserdar 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 labels Jul 25, 2023
@v-mserdar v-mserdar added this to the S120 milestone Jul 25, 2023
@v-mserdar v-mserdar requested a review from a team as a code owner July 25, 2023 22:16
SergeyGaluzo
SergeyGaluzo approved these changes Jul 25, 2023
View reviewed changes
Copy link
Contributor

@SergeyGaluzo SergeyGaluzo left a comment

Choose a reason for hiding this comment

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

:shipit:

@v-mserdar v-mserdar merged commit 1591241 into main Jul 31, 2023
@v-mserdar v-mserdar deleted the personal/v-mserdar/sqlretry_update branch July 31, 2023 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SergeyGaluzo SergeyGaluzo SergeyGaluzo approved these changes

@PTaladay PTaladay PTaladay approved these changes

Assignees
No one assigned
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.
Projects
None yet
Milestone
S120
Development

Successfully merging this pull request may close these issues.
3 participants
@v-mserdar @PTaladay @SergeyGaluzo