Description

Story: https://breedinginsight.atlassian.net/browse/BI-2058

Added Observations and ObservationUnits to the redis cache. They are eagerly fetched on application startup just like the other entities we cache.

All the methods that write Observations or ObservationUnits have been modified to repopulate the cache, which is essential for reading one's own writes.

Most of the methods that read Observations and Observation units have been modified to read from the cache.

A few read methods still make BrAPI calls because it would be impractical to use the cache. For example, I did not update BrAPIObservationUnitDAO::getObservationUnits because some of the search parameters would be expensive to implement client-side (compared to SQL joins), notably the seasonDbId parameter, which is not directly on the ObservationUnit object.

Also, calling ProgramCache::post with a null postFunction would result in a NPE, I prevented this possibility, see this commit.

Dependencies

bi-web develop branch

Testing

Ensure experiment upload, view, and download (file export) workflows work as expected.

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>