-
Notifications
You must be signed in to change notification settings - Fork 247
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
RoundhousE does not update its own Tables #16
Comments
Unfortunately it's a limitation of nh that it doesn't change existing columns very well. |
What we can do is add something to explicitly make the change to the table if the size is not correct. |
Cool, I will get around to taking a look at this. |
sorry on the close, finger fumble |
They shouldn't be so close together...lol |
Originally the thought was to go with NHibernate and Fluent NHibernate because they would provide the most surface into internal upgrades. We may want to explore options here at some point, but we can add a database specific upgrade task for now to handle when we do things that nh doesn't pick up. |
Bit ironic really :)
We started with an old version of RH on our servers. Now when we get errors, we get the barf that RH cannot insert into the errors table (message is truncated).
Looks like Message field in the ScriptRunErrors table was changed from nvarchar(255) to ntext a while back, but RH doesn't version its own tables so we get the log barf instead of the real error message.
I can take a look at this, unfortunately, it may lead to a state of infinite recursion.
The text was updated successfully, but these errors were encountered: