Allow reindex with unlimited empty pages

mikaelweave commented 1 month ago

Description

Removes artificial limitation of reindex where there is large historical datasets or customer wants limited query size.

Customer has tested my reindex fix from the last release and it's not quite right. It doesn't allow for many sequential historical/deleted resources with default query size. The reindex still says completed incorrectly in this case. This change adds an outer loop to continue getting surrogate ranges until there is one w/ data.

Related issues

AB#124883

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.
[ ] CI is green before merge
Review squash-merge requirements

Semver Change (docs)

Patch|Skip|Feature|Breaking (reason)

mikaelweave commented 1 month ago

/azp run

azure-pipelines[bot] commented 1 month ago

Azure Pipelines successfully started running 1 pipeline(s).

mikaelweave commented 1 month ago

/azp run

azure-pipelines[bot] commented 1 month ago

Azure Pipelines successfully started running 1 pipeline(s).

mikaelweave commented 1 month ago

/azp run

azure-pipelines[bot] commented 1 month ago

Azure Pipelines successfully started running 1 pipeline(s).

mikaelweave commented 1 month ago

/azp run

azure-pipelines[bot] commented 1 month ago

Azure Pipelines successfully started running 1 pipeline(s).

microsoft / fhir-server