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.

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

Allow Setting the Database Sync Timeout #566

Merged
merged 1 commit into from
Dec 14, 2021
Merged

Conversation

alexanderkiel
Copy link
Member

@alexanderkiel alexanderkiel commented Dec 14, 2021

DB_SYNC_TIMEOUT - Timeout in milliseconds for all reading FHIR interactions acquiring the newest database state.

All reading FHIR interactions have to acquire the last database state known at the time the request arrived in order to ensure consistency. That database state might not be ready immediately because indexing might be still undergoing. In such a situation, the request has to wait for the database state becoming available. If the database state won't be available before the timeout expires, a 503 Service Unavailable response will be returned. Please increase this timeout if you experience such 503 responses, and you are not able to improve indexing performance or lower transaction load.

@alexanderkiel alexanderkiel self-assigned this Dec 14, 2021
@alexanderkiel alexanderkiel added the operation Operational issues like monitoring and maintenance. label Dec 14, 2021
@alexanderkiel alexanderkiel added this to the v0.15 milestone Dec 14, 2021
All reading FHIR interactions have to acquire the last database state
known at the time the request arrived in order to ensure consistency.
That database state might not be ready immediately because indexing
might be still undergoing. In such a situation, the request has to wait
for the database state becoming available. If the database state won't
be available before the timeout expires, a 503 Service Unavailable
response will be returned. Please increase this timeout if you
experience such 503 responses, and you are not able to improve indexing
performance or lower transaction load.
@codecov
Copy link

codecov bot commented Dec 14, 2021

Codecov Report

Merging #566 (ab3e264) into develop (82e1da9) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #566   +/-   ##
========================================
  Coverage    92.43%   92.43%           
========================================
  Files          229      229           
  Lines        13599    13601    +2     
  Branches       407      407           
========================================
+ Hits         12570    12572    +2     
  Misses         622      622           
  Partials       407      407           
Impacted Files Coverage Δ
modules/rest-api/src/blaze/rest_api.clj 89.06% <100.00%> (+0.17%) ⬆️
modules/rest-api/src/blaze/rest_api/routes.clj 99.35% <100.00%> (ø)
modules/rest-util/src/blaze/middleware/fhir/db.clj 100.00% <100.00%> (ø)

@alexanderkiel alexanderkiel merged commit ed62af4 into develop Dec 14, 2021
@alexanderkiel alexanderkiel deleted the read-timeout branch December 14, 2021 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
operation Operational issues like monitoring and maintenance.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant