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

Fix performance for metadata url #3274

Merged
merged 2 commits into from May 2, 2023
Merged

Fix performance for metadata url #3274

merged 2 commits into from May 2, 2023

Conversation

abiisnn
Copy link
Contributor

@abiisnn abiisnn commented May 2, 2023 •
edited

Description

Fix performance issue mentioned in my last PR.

Related issues

Addresses [issue #].

Testing

  • Access the /metadata endpoint.
    image

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)

@abiisnn abiisnn mentioned this pull request May 2, 2023
Merged
1 task
@abiisnn abiisnn added this to the S114 milestone May 2, 2023
@abiisnn abiisnn added the Open source This change is only relevant to the OSS code or release. label May 2, 2023
@abiisnn abiisnn marked this pull request as ready for review May 2, 2023 02:04
@abiisnn abiisnn requested a review from a team as a code owner May 2, 2023 02:04
@abiisnn abiisnn added the Enhancement Enhancement on existing functionality. label May 2, 2023
PTaladay
PTaladay previously approved these changes May 2, 2023
View reviewed changes
brendankowitz
brendankowitz approved these changes May 2, 2023
View reviewed changes
Copy link
Member

@brendankowitz brendankowitz left a comment

Choose a reason for hiding this comment

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

:shipit:

@abiisnn abiisnn merged commit b48bf36 into main May 2, 2023
7 checks passed
@abiisnn abiisnn deleted the abiisnn/fix_metadata_performance branch May 2, 2023 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brendankowitz brendankowitz brendankowitz approved these changes

@PTaladay PTaladay PTaladay left review comments

Assignees
No one assigned
Labels
Enhancement Enhancement on existing functionality. Open source This change is only relevant to the OSS code or release.
Projects
None yet
Milestone
S114
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@abiisnn @brendankowitz @PTaladay