mitodl / mitodl/ol-data-platform

Migrate `instructor_module_report` to use dimensional layer (blocked by mart migration)

Open
#2,115 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement product:data-platform
Dominant language
Python
Stars
51
Forks
6
Avg merge
2d 19h
Merged PRs (30d)
113

Description

Epic: #2072

File: src/ol_dbt/models/reporting/instructor_module_report.sql

Problem: References marts__combined_course_enrollment_detail and the organization_administration_report reporting model (which itself bypasses). Blocked by migration of the mart and the reporting model.

Note: This issue is blocked by the migration of the upstream mart model(s) it references. It should be addressed after those mart migrations are complete.

Acceptance criteria:

  • Model references only dimensional/, mart, or reporting models (all of which use dimensional)
  • No {{ ref('int__*') }} or {{ ref('stg__*') }} calls remain in this model or its upstream mart dependencies
  • Model compiles and passes dbt tests

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

After the upstream mart and reporting-model migrations are complete, start with src/ol_dbt/models/reporting/instructor_module_report.sql and inspect its references to marts__combined_course_enrollment_detail and organization_administration_report. Confirm that upstream dependencies no longer bypass the dimensional layer, then run dbt compilation and tests. Done means the model and its upstream mart dependencies contain no int__* or stg__* refs and pass dbt tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
sql
Domain
data-engineering
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.