HicServices / RDMP / 26280548007

57%

push

github

Cohort Builder: partial-refresh AggregateConfiguration renames/moves (#2336)

Adds AggregateConfiguration to the SelectiveRefresh type switch in
CatalogueChildProvider.SelectiveRefresh(IMapsDirectlyToDatabaseTable).
Previously this type fell through to '_ => false', forcing
ActivateItems.RefreshBus_RefreshObject down the full GetChildProvider()
rebuild path (~9-10s at HIC scale) on every aggregate rename, parameter
edit, or other property change.

Implements SelectiveRefresh(AggregateConfiguration ac) following the
established pattern from SelectiveRefresh(CohortAggregateContainer):

  - For cohort builder set members (parent CohortAggregateContainer):
    re-fetch aggregates + container hierarchy, refresh the parent
    container's subtree.

  - For graph aggregates hanging off a Catalogue: re-fetch aggregates,
    refresh the catalogue's subtree.

  - Falls back to false (i.e. full rebuild) if neither parent type is
    in the descendancy.

Pairs with the preceding 'inject joinable knowledge after partial-refresh
rebuild' commit on this branch — that fix is what makes the
BuildAggregateConfigurations() call here cheap rather than triggering
N database round-trips downstream.

Expected impact on HIC platform: aggregate rename freeze drops from
~10s to ~1-2s (matching the now-snappy container rename).

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

11550 of 21765 branches covered (53.07%)

Branch coverage included in aggregate %.

0 of 19 new or added lines in 1 file covered. (0.0%)

32678 of 55687 relevant lines covered (58.68%)

18092.04 hits per line