Repository navigation
[BI-2923] Associate observation variables with brapi program db id on creation - #571
jloux-brapi wants to merge 1 commit into
Conversation
| // Get brapi variables | ||
| VariableQueryParams variablesRequest = new VariableQueryParams(); | ||
| variablesRequest.externalReferenceSource(referenceSource); | ||
| variablesRequest.programDbId(brapiProgramDbId); |
There was a problem hiding this comment.
since traits are now fetched only by programDbId, any variable with a NULL program_id will trigger the throw at line 211 for the whole program. The only way I can see that happening is if the ProdServer migration runs first and the old bi-api creates a trait before this version is deployed. Will these three PRs always be deployed together? If not, it might be worth falling back to the exref lookup for any trait IDs missing from the result.
There was a problem hiding this comment.
Yes, the plan is deploy all of these PRs together.
I think this release will be major enough we can take down the sites for a bit of time so that corner case cannot happen. @nickpalladino can give his two cents.
Another option is to make programDbId mandatory for the prod server side, which would mean a failure if it's not submitted in a creation request. That said, none of the other cache entities have program as non null so I think it's relatively safe to move forward.
There was a problem hiding this comment.
Thanks, the coordinated deployment addresses the rollout concern.
Description
Story: BI-2923
Dependencies
Testing
Ontology PageShow Detailsand then clickingEdit, where if they are not already used you can edit some elements of the term. Activating and Deactivating also updates ontology. Simplify verify the changes are reflected once they are made.Checklist: