Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to pihome_mysql_database.sql #66

Merged
merged 5 commits into from Sep 17, 2019
Merged

Conversation

twa127
Copy link
Contributor

@twa127 twa127 commented Sep 16, 2019

Changes to match the mysqldump output format. All numeric DEFAULT values enclosed in single quotes, TINYTEXT fields do not accept a DEFAULT value, timestamp() changed to TIMESTAMP and changes to the COLLATE format in mqtt and nodes tables.

All in preparation for an update database schema script.

Changes to make compatible with mysqldump format, all numeric default values enclosed in single quotes, tinyint fields should not have a default value set and TIMESTAMP format changed.
Missed the DEFAULT values in table mqtt
@pihome-shc
Copy link
Owner

@twa127
what are you using to export mysql database? i m using open-source HeidiSQL windows client to connect to mysql.
All exports are through this software, if there is better option i dont mind switching.

@twa127
Copy link
Contributor Author

twa127 commented Sep 16, 2019

Using mysqldump with the -d switch and then stripping out the views and triggers to produce an sql file I can compare against pihome_mysql_database.sql

@pihome-shc pihome-shc merged commit 8d80459 into pihome-shc:master Sep 17, 2019
twa127 referenced this pull request in twa127/pihome Sep 18, 2019
Merge pull request #66 from twa127/master
pihome-shc pushed a commit that referenced this pull request Mar 21, 2020
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.

None yet

2 participants