-
Notifications
You must be signed in to change notification settings - Fork 13.8k
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
Adding in Second and Minute time grains for MySQL #696
Conversation
… to the models for Dashboards or Slices (using declarative base)
@mistercrunch - as part of the work around the autorefresh settings for the dashboards I've also fixed one of the migrations to now use declarative base. Consider it a seperate commit, but will need to be merged in preparation for the autorefresh work. |
Which database you using? I'm still confused as to why migrations are working in the context of travis but aren't working for some users. The declarative base thing should not be necessary from my understanding. |
I'm using mysql. The migration works fine normally. When I add new fields When applying the migration I've edited (before editing it) I get a SQL Also, if I'm doing something ridiculous though, let me know. I'm pretty On Wed, 29 Jun 2016 21:09 Maxime Beauchemin, notifications@github.com
Alan Cruickshank |
So when you install a new version you then run |
@mistercrunch if I add two columns into the dashboard model as follows:
then here's the stack trace I get if I don't change the migration:
|
What about you do, in order:
|
…ds added to the models for Dashboards or Slices (using declarative base)" This reverts commit 0b6dd69.
@mistercrunch I'll try what you suggested with the migrations and if I still need to make changes I'll submit them in a seperate PR. In the mean time, it's probably still worth merging in the new MySQL time grains. |
@mistercrunch I went through the steps you suggested and they do successfully create the new column in the DB. However, when I then clean everything and run through the migrations from scratch, I get to the same error (stacktrace below). Migration
|
Adding in Second and Minute time grains for MySQL