Skip to content

docs: Upload star schema#14

Open
Loc-04 wants to merge 1 commit into
devfrom
task/star-schema-erd
Open

docs: Upload star schema#14
Loc-04 wants to merge 1 commit into
devfrom
task/star-schema-erd

Conversation

@Loc-04
Copy link
Copy Markdown
Collaborator

@Loc-04 Loc-04 commented May 28, 2026

This pull request include the star schema ERD

Copilot AI review requested due to automatic review settings May 28, 2026 07:18
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@perinst
Copy link
Copy Markdown
Collaborator

perinst commented May 28, 2026

I've seen about this ERD but, I quite concerned about how to you find reason to choose 3 fact table and other Dim tables, Any document instruction about way do define before draw ERD and purpose of tables,

@Loc-04
Copy link
Copy Markdown
Collaborator Author

Loc-04 commented May 28, 2026

I've seen about this ERD but, I quite concerned about how to you find reason to choose 3 fact table and other Dim tables, Any document instruction about way do define before draw ERD and purpose of tables,

Hi, thanks for your question. To decide the grain of each fact table, i followed the dashboard mockup reference and based on the requirement of each dashboard from
Rookie Data - Phase 2.pdf

A fact is determined if the dashboard ask what (count/sum/avg) and what in the filter will be the 'context' for that fact (by region, agent, etc). Please give any suggestion if you feel this still not enough or wrong based on Kimball standard 👍

@Loc-04
Copy link
Copy Markdown
Collaborator Author

Loc-04 commented May 28, 2026

I've seen about this ERD but, I quite concerned about how to you find reason to choose 3 fact table and other Dim tables, Any document instruction about way do define before draw ERD and purpose of tables,

Read this pr first before viewing the ERD Schema : #12 (comment)

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