Open DarrenWHenderson opened 3 years ago
@jklann raises an important issue with how the check for a non-demographic fact since 2012 was being performed here: https://github.com/i2b2plugins/loyalty_cohort/blob/2c0be0326d4a5dea6c0bc457da1f07c12ee26304/OPTIMIZATION_REFACTOR_V1.sql#L77
A quick temp table to prior to this step to grab all our non-demographic concepts based off paths in concept dimension, and a rewrite on this line to join to this temp table works, and did not increase execution time on my production environment.
@jklann raises an important issue with how the check for a non-demographic fact since 2012 was being performed here: https://github.com/i2b2plugins/loyalty_cohort/blob/2c0be0326d4a5dea6c0bc457da1f07c12ee26304/OPTIMIZATION_REFACTOR_V1.sql#L77
A quick temp table to prior to this step to grab all our non-demographic concepts based off paths in concept dimension, and a rewrite on this line to join to this temp table works, and did not increase execution time on my production environment.