Closed Bug 1007994 Opened 8 years ago Closed 8 years ago

Add MySQL specific database schema dumps to the BuildAPI repo

Categories

(Release Engineering :: General, defect)

Product:
Release Engineering
Component:
General
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: zeller, Unassigned)

Details

Attachments

(1 file, 1 obsolete file)

mysql-schemas.patch
25.05 KB, patch
hwine
: review+
hwine
: checked-in+
Details | Diff | Splinter Review
Attached patch mysql-schemas.patch (obsolete) — — Splinter Review 
Currently there are only sqlite schema dumps in the buildapi repo. We should add mysql specific dumps for the time being. Ultimately, it would be nice to have abstract schemas to load with something like sqlalchemy into any db we would like.
Attachment #8419806 - Flags: review?(hwine)
Comment on attachment 8419806 [details] [diff] [review]
mysql-schemas.patch

Review of attachment 8419806 [details] [diff] [review]:
-----------------------------------------------------------------

Good on updating the README.txt as well :)  Take it one more step an include a note on how you generated the schema files, so it's easy for the next person to update.
Attachment #8419806 - Flags: review?(hwine) → review+
Added README text detailing how schemas were obtained
Attachment #8419806 - Attachment is obsolete: true
Attachment #8419828 - Flags: review?(hwine)
Comment on attachment 8419828 [details] [diff] [review]
mysql-schemas.patch

Review of attachment 8419828 [details] [diff] [review]:
-----------------------------------------------------------------

r+ on the README additions; <stamp/> on the schemas since they're generated.
Attachment #8419828 - Flags: review?(hwine) → review+
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.