Skip to content

Commit 4e531d4

Browse files
committed
perf(sql): use org and course_key filters to use primary key indexes
1 parent 20b4f53 commit 4e531d4

30 files changed

+27
-456
lines changed

tutoraspects/templates/openedx-assets/queries/active_last_7_days.sql

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,13 @@ with
22
recent_activity as (
33
select course_key, COUNT(DISTINCT actor_id) as active_last_7_days
44
from {{ ASPECTS_XAPI_DATABASE }}.navigation_events
5-
where emission_time >= NOW() - INTERVAL 7 DAY
5+
where
6+
emission_time >= NOW() - INTERVAL 7 DAY
7+
{% include 'openedx-assets/queries/common_filters.sql' %}
68
group by course_key
79
)
810

911
select fss.*, COALESCE(ra.active_last_7_days, 0) as active_within_last_7_days
1012
from {{ DBT_PROFILE_TARGET_DATABASE }}.fact_student_status fss
1113
left join recent_activity ra on fss.course_key = ra.course_key
14+
where 1 = 1 {% include 'openedx-assets/queries/common_filters.sql' %}

tutoraspects/templates/openedx-assets/queries/at_risk_learner_filter.sql

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,4 +12,6 @@ with
1212
select org, course_key, learners.actor_id as actor_id
1313
from {{ DBT_PROFILE_TARGET_DATABASE }}.fact_student_status learners
1414
join page_visits using (org, course_key, actor_id)
15-
where approving_state = 'failed' and enrollment_status = 'registered'
15+
where
16+
approving_state = 'failed' and enrollment_status = 'registered'
17+
{% include 'openedx-assets/queries/common_filters.sql' %}

tutoraspects/templates/openedx-assets/queries/at_risk_problem_results.sql

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,3 +4,4 @@ join
44
(
55
{% include 'openedx-assets/queries/at_risk_learner_filter.sql' %}
66
) as at_risk_learners using (org, course_key, actor_id)
7+
where 1 = 1 {% include 'openedx-assets/queries/common_filters.sql' %}

tutoraspects/templates/openedx-assets/queries/dim_at_risk_learners.sql

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,3 +29,4 @@ where
2929
approving_state = 'failed'
3030
and enrollment_status = 'registered'
3131
and page_visits.last_visited < subtractDays(now(), 7)
32+
{% include 'openedx-assets/queries/common_filters.sql' %}

tutoraspects/templates/openedx-assets/queries/dim_course_problems.sql

Lines changed: 0 additions & 20 deletions
This file was deleted.

tutoraspects/templates/openedx-assets/queries/dim_course_videos.sql

Lines changed: 0 additions & 20 deletions
This file was deleted.

tutoraspects/templates/openedx-assets/queries/enrollment_status.sql

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,3 +12,4 @@ left join
1212
{{ ASPECTS_EVENT_SINK_DATABASE }}.course_names cn
1313
on fes.org = cn.org
1414
and fes.course_key = cn.course_key
15+
where 1 = 1 {% include 'openedx-assets/queries/common_filters.sql' %}

tutoraspects/templates/openedx-assets/queries/fact_at_risk_navigation_completion.sql

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,3 +4,4 @@ join
44
(
55
{% include 'openedx-assets/queries/at_risk_learner_filter.sql' %}
66
) as at_risk_learners using (org, course_key, actor_id)
7+
where 1 = 1 {% include 'openedx-assets/queries/common_filters.sql' %}

tutoraspects/templates/openedx-assets/queries/fact_at_risk_pageview_engagement.sql

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,3 +4,4 @@ join
44
(
55
{% include 'openedx-assets/queries/at_risk_learner_filter.sql' %}
66
) as at_risk_learners using (org, course_key, actor_id)
7+
where 1 = 1 {% include 'openedx-assets/queries/common_filters.sql' %}

tutoraspects/templates/openedx-assets/queries/fact_at_risk_problem_engagement.sql

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
1-
{% include 'openedx-assets/queries/fact_problem_engagement.sql' %}
1+
with fact_problem_engagement as
2+
({% include 'openedx-assets/queries/fact_problem_engagement.sql' %})
3+
select fact_problem_engagement.* from fact_problem_engagement pe
24
join
35
(
46
{% include 'openedx-assets/queries/at_risk_learner_filter.sql' %}

tutoraspects/templates/openedx-assets/queries/fact_at_risk_video_engagement.sql

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,3 +4,4 @@ join
44
(
55
{% include 'openedx-assets/queries/at_risk_learner_filter.sql' %}
66
) as at_risk_learners using (org, course_key, actor_id)
7+
where 1 = 1 {% include 'openedx-assets/queries/common_filters.sql' %}

tutoraspects/templates/openedx-assets/queries/fact_at_risk_video_plays.sql

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,3 +6,4 @@ join
66
(
77
{% include 'openedx-assets/queries/at_risk_learner_filter.sql' %}
88
) as at_risk_learners using (org, course_key, actor_id)
9+
where 1 = 1 {% include 'openedx-assets/queries/common_filters.sql' %}

tutoraspects/templates/openedx-assets/queries/fact_at_risk_video_watches.sql

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,3 +6,4 @@ join
66
(
77
{% include 'openedx-assets/queries/at_risk_learner_filter.sql' %}
88
) as at_risk_learners using (org, course_key, actor_id)
9+
where 1 = 1 {% include 'openedx-assets/queries/common_filters.sql' %}

tutoraspects/templates/openedx-assets/queries/fact_at_risk_watched_video_segments.sql

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,3 +6,4 @@ join
66
(
77
{% include 'openedx-assets/queries/at_risk_learner_filter.sql' %}
88
) as at_risk_learners using (org, course_key, actor_id)
9+
where 1 = 1 {% include 'openedx-assets/queries/common_filters.sql' %}

tutoraspects/templates/openedx-assets/queries/fact_forum_interactions.sql

Lines changed: 0 additions & 3 deletions
This file was deleted.

tutoraspects/templates/openedx-assets/queries/fact_learner_problem_course_summary.sql

Lines changed: 0 additions & 136 deletions
This file was deleted.

tutoraspects/templates/openedx-assets/queries/fact_learner_problem_summary.sql

Lines changed: 0 additions & 125 deletions
This file was deleted.

tutoraspects/templates/openedx-assets/queries/fact_learner_summary.sql

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,3 +66,4 @@ left join
6666
on fss.org = let.org
6767
and fss.course_key = let.course_key
6868
and fss.actor_id = let.actor_id
69+
where 1 = 1 {% include 'openedx-assets/queries/common_filters.sql' %}

tutoraspects/templates/openedx-assets/queries/fact_page_engagement.sql

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,3 +50,4 @@ join
5050
left outer join
5151
{{ DBT_PROFILE_TARGET_DATABASE }}.dim_user_pii users
5252
on toUUID(pv.actor_id) = users.external_user_id
53+
where 1 = 1 {% include 'openedx-assets/queries/common_filters.sql' %}

tutoraspects/templates/openedx-assets/queries/fact_problem_engagement.sql

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,3 +51,4 @@ join
5151
left outer join
5252
{{ DBT_PROFILE_TARGET_DATABASE }}.dim_user_pii users
5353
on toUUID(pe.actor_id) = users.external_user_id
54+
where 1 = 1 {% include 'openedx-assets/queries/common_filters.sql' %}

0 commit comments

Comments
 (0)