Define the term as 'schema dimension' (shortened to 'dimension') - #239
Conversation
Promote 'schema dimension' to the defined term in its two definitional homes (Entity Integrity section Schema dimensions; Read Diagrams section Dimensions and Underlined Names), then shorten to 'dimension' in-context. Canon already used 'Schema dimensions' as the section heading and 'lineage through schema dimensions' in prose — this makes the defined term consistent and disambiguates from array and data-warehouse 'dimension'. Headings/anchors unchanged. Note: read-diagrams cell source normalized to a single string by the notebook editor; llms-full.txt regenerates from source.
MilagrosMarin
left a comment
There was a problem hiding this comment.
The dimension → schema dimension promotion is well-motivated (the collision with warehouse dimension tables and array axes is real, and the Entity Integrity page already titled the section ## Schema dimensions), and the "define once, shorten in-context" approach keeps all headings and anchors stable. CI green.
Three mechanical notes on the notebook, none about the wording itself:
-
Trailing newline dropped — the diff ends with
\ No newline at end of file. Every other notebook in the repo (model-relationships,02-schema-design,master-part, andread-diagramsonmain) ends with one, so this is the only file without. -
The cell-source normalization is worth undoing. You flag it as benign nbformat, and it is — but on
mainall 54 cells inread-diagrams.ipynbuse list-of-stringssource; on this branch that cell is now the only plain-string one. The practical cost is that any future edit to that Markdown cell diffs as a single unsplittable line rather than per-line — which matters for a cell likely to be touched again. -
Parenthetical style differs between the two definitional homes —
entity-integrity.mduses em-dashes (— often shortened to *dimension* —), the notebook uses parens ((often shortened to *dimension*)). Same meaning, just reads as drift given both land in this PR.
Approving — none of these are correctness issues.
Promotes schema dimension to the defined term, then shortens to dimension in-context.
Why
dimensionis overloaded — data-warehouse dimension tables, OLAP cubes, and array/tensor dimensions (the type-system and npy pages already use "dimension" for array axes). Qualifying it as schema dimension on definition disambiguates cleanly. This isn't a new coinage: the Entity Integrity page already titles the section## Schema dimensionsand says "lineage through schema dimensions" — this just makes the defined term consistent with that.Changes (definitional homes only)
A **dimension** is…→A **schema dimension** — often shortened to *dimension* — is…Notes
llms-full.txtregenerates from source (not hand-edited here).sourcewas normalized to a single JSON string by the notebook editor (valid nbformat, identical rendering).