Legend:
- Unmodified
- Added
- Removed
-
docs/P2-RelationalDesign/wiki/RelationalDesignAIUsage.md
r0cee8ec r1549dae 12 12 === Diagram === 13 13 14 The student produces `relational_schema.jpg` in DBeaver from the live `project` schema; see [wiki:RelationalDesign]for instructions.14 `relational_diagram_v4.png` was exported by the student in DBeaver from the live `project` schema, with the tables in the same positions as the entity sets of `ERModel_v05.png`; see [wiki:RelationalDesign] (section "How to regenerate it") for instructions. 15 15 16 16 === Results in details / description === … … 96 96 `trade.go` alone to keep the reservation consistent — the same reasoning 97 97 already applied to `avg_price NOT NULL` in session 2. 98 99 === Session 4 — 2026-09-29 (Claude Opus 5.5): correction after review === 100 101 '''Prompt (student, excerpt):''' I pasted the reviewer's comments on P2: 102 103 > the `contains` relationship from the model is implemented differently in the database 104 > the `holds` relationship from the model is implemented differently in the database 105 > there may be other inconsistencies that are difficult to spot; the Phase 2 diagram should 106 > have been created using DBeaver with a layout visually similar to the ERD 107 108 > I suggest correcting only the Phase 1 model so that it matches Phase 2 [...] For Phase 2, 109 > simply update the image according to the requirements. 110 111 '''Response (AI, summarised):''' 112 113 * Explained the mismatch. In the model, `Holds` and `Contains` were M:N relationships, whose table key would be the pair `{user_id, crypto_id}` / `{watchlist_id, crypto_id}`. But `holdings` and `watchlist_items` have their own `id` primary key, which is how an entity set is implemented. Following the reviewer's advice, P1 was corrected (v05, entity sets `Holdings` and `WatchlistItems`), and the database was not changed. 114 * Found the other inconsistencies between this page and the live schema. The page was missing `users.reserved_balance`, `orders.filled_quantity`, the status `partially_filled`, `market_trades.buy_order_id` / `sell_order_id` and the whole `order_events` table. The "10 tables", "5 indexes" and "the one nullable foreign key" counts were also out of date (really 11 tables, 8 indexes in `schema_creation.sql`, and 3 nullable foreign keys). 115 * Rewrote [wiki:RelationalDesign]. Each relation is labelled with its entity set and each foreign key with its relationship, and the transformation is a table of all 15 relationships → 15 foreign keys, with `NOT NULL` following participation. 116 * Wrote export instructions with a table grid that mirrors `ERModel_v05.png`. 117 118 '''What I decided:''' to correct P1 instead of the database, as the reviewer suggested. I 119 exported `relational_diagram_v4.png` from DBeaver myself, with the tables arranged like the 120 ER diagram.
Note:
See TracChangeset
for help on using the changeset viewer.
