Skip to content

[BI-1193-2] BI-API throws timeout exception when BB mat view refresh exceeds 10m#258

Merged
dmeidlin merged 1 commit intodevelopfrom
bug/BI-1193-2
May 31, 2023
Merged

[BI-1193-2] BI-API throws timeout exception when BB mat view refresh exceeds 10m#258
dmeidlin merged 1 commit intodevelopfrom
bug/BI-1193-2

Conversation

@dmeidlin
Copy link
Contributor

Description

Importing of experiments was failing during QA because the volume of data (for example, ~100,00 germplasm) was causing the time to refresh the phenotype materialized view after obs. unit creation to exceed the limit set in bi-api (10 minutes) when a timeout exception is thrown if it hasn't received a response from the brapi service.

The long-term solution is to update the Breedbase code to query the db directly instead of relying on mat. views.

The short-term solution here is to expose an environment variable that can be used to easily set the timeout value for bi-api. Then it can be adjusted as needed for different test or production environments.

Dependencies

breedinginsight/breedbase fork running bug/BI-1193 branch

Testing

Please include any details needed for reviewers to test this code

Checklist:

  • I have performed a self-review of my own code
  • I have tested my code and ensured it meets the acceptance criteria of the story
  • I have tested that my code works with both the brapi-java-server and BreedBase
  • I have create/modified unit tests to cover this change
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to documentation
  • I have run TAF: <please include a link to TAF run>

@dmeidlin dmeidlin requested a review from timparsons May 31, 2023 19:55
@github-actions github-actions bot added the bug Something isn't working label May 31, 2023
@dmeidlin dmeidlin requested a review from nickpalladino May 31, 2023 19:55
@dmeidlin dmeidlin merged commit 6c30da2 into develop May 31, 2023
@dmeidlin dmeidlin deleted the bug/BI-1193-2 branch May 31, 2023 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants