Adding stored procedure instead of UNION#5300

Merged

SergeyGaluzo merged 32 commits intomainfrom

users/sergal/splayer0

Jan 6, 2026

Contributor

SergeyGaluzo commented Jan 1, 2026 •

edited

Loading

Fixes https://microsofthealth.visualstudio.com/Health/_workitems/edit/178162

SergeyGaluzo added 2 commits

December 30, 2025 16:09


          new SQL version

d28c550


          Pretty

5b49183

SergeyGaluzo requested a review from a team as a code owner

January 1, 2026 22:39

github-actions bot added the SQL Scripts label

SergeyGaluzo added 5 commits

January 1, 2026 14:42


          no order by type

b82ee75


          Remove version switch from TokenQueryGenerator

fc4473a


          schema switch

323127f


          Removed dead code left by Paul

6a0cb31


          Added overflow handling

da989db

SergeyGaluzo changed the title ~~Adding stored procedure instead of UNION bug~~ Adding stored procedure instead of UNION

SergeyGaluzo added 11 commits

January 1, 2026 16:53


          tweaks

ceb42df


          Reformatted stored proc

b9a53c4


          more tweaks

e682ce7


          simpler

b2c9778


          TypeOf

ab57bc8


          minus usings

9d09df6


          private

a9f8ca5


          Token tests

9bf3835


          Corrected but not final

93c2b1b


          complete tests


          common logic

microsoft deleted a comment from azure-pipelines bot

SergeyGaluzo added 8 commits

January 4, 2026 10:18


          Added search checks

488ea54


          simple wait loop back

e240bbb


          One more test

cc3c0c9


          Adding cache misses

77ba6e5


          Data row checks

678cfc2


          derive search param from url in tests

aa85a08


          nicer sql

94c85d9


          nicer constructor

32b9095

LTA-Thinking reviewed

View reviewed changes

src/Microsoft.Health.Fhir.SqlServer/Features/Search/SqlServerSearchService.cs Outdated Show resolved Hide resolved

LTA-Thinking reviewed

View reviewed changes

src/Microsoft.Health.Fhir.SqlServer/Features/Search/SqlServerSearchService.cs Show resolved Hide resolved


          null checks inside

261bb9e

LTA-Thinking reviewed

View reviewed changes

src/Microsoft.Health.Fhir.SqlServer/Features/Schema/Sql/Sprocs/GetResourcesByTokens.sql Show resolved Hide resolved

LTA-Thinking reviewed

View reviewed changes

src/Microsoft.Health.Fhir.SqlServer/Features/Schema/Sql/Sprocs/GetResourcesByTokens.sql Outdated Show resolved Hide resolved

LTA-Thinking reviewed

View reviewed changes

test/Microsoft.Health.Fhir.Shared.Tests.Integration/Persistence/SqlComplexQueryTests.cs Outdated Show resolved Hide resolved

SergeyGaluzo added 3 commits

January 5, 2026 10:16


          tests and better try

263266a


          6 resources

3689e9b


          Remove redundant history and deleted clauses

0054c9c

fhibf previously approved these changes

View reviewed changes


          truncation 128 logic

d8634d5

SergeyGaluzo dismissed fhibf’s stale review via

d8634d5

January 6, 2026 00:40


          Added code overflow check

c0f7a74

microsoft deleted a comment from azure-pipelines bot

LTA-Thinking approved these changes

View reviewed changes

SergeyGaluzo added Azure Healthcare APIs Schema Version backward compatible labels

SergeyGaluzo added this to the CY25Q3/2Wk07 milestone

rajithaalurims approved these changes

View reviewed changes

SergeyGaluzo merged commit 5c9aec7 into main

59 of 63 checks passed

SergeyGaluzo deleted the users/sergal/splayer0 branch

January 6, 2026 19:09

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

Labels

Azure Healthcare APIs Schema Version backward compatible SQL Scripts