Skip to content

Conversation

@alisman
Copy link
Contributor

@alisman alisman commented Nov 18, 2025

No description provided.

sheridancbio and others added 3 commits November 18, 2025 17:17
- changes to reduce memory footprint during derivation
    - clinical event table : 4 data types written separately (rather than UNION)
    - JOIN of clinical_event to clinical_event_data rearranged : clinical_event_data now referenced first
- changes to reduce runtime during query operations
    - clinical event table : re-order table part storage order (primary key)
        move genetic_profile_stable_id from rank #4 to rank #1, and add cancer_study_identifier as #2
…-only

# Conflicts:
#	src/main/resources/db-scripts/clickhouse/clickhouse.sql
@alisman alisman changed the base branch from master to rc-7.0-clickhouse-only November 18, 2025 22:41
@alisman alisman marked this pull request as ready for review November 18, 2025 22:42
@sheridancbio
Copy link
Contributor

I've looked over the changes here (bringing in the changes which are now in master into the rc-7.0-clickhouse-only branch)
This looks valid to me. schema numbers have been properly updated in clickhouse.sql and pom.xml, and needed changes are present. There may be some unneeded / non-standard whitespace, but I think this is usable and I will be using this changeset for the next msk portal import cycle.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants