-
Notifications
You must be signed in to change notification settings - Fork 579
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
[dev.icinga.com #6646] NULL vs empty string #1757
Comments
Updated by mfriedrich on 2014-07-06 10:00:38 00:00
|
Updated by mfriedrich on 2014-07-09 15:04:35 00:00
|
Updated by mfriedrich on 2014-07-10 15:19:05 00:00
|
Updated by gbeutner on 2014-07-11 09:23:31 00:00
Required changes:
|
Updated by gbeutner on 2014-07-22 08:21:44 00:00
I'm limiting the scope of the change to the icinga_objects table so as to not break other tables which might not have default values for some of their columns. What this means is that after this is fixed new rows in the icinga_objects table should have name2 set to whatever is the default value for that column (apparently NULL atm). |
Updated by gbeutner on 2014-07-22 08:30:04 00:00
Applied in changeset i2:f86068d5e089b75b4b1bafe31829d9992d26d1ad. |
Updated by tgelf on 2014-07-22 10:51:56 00:00
Please also take care of "dirty" tables for existing Icinga 2.0 stable setups. Thanks, |
Updated by gbeutner on 2014-07-23 08:30:05 00:00
Applied in changeset i2:1a361d834ade415957a6b7f7f38e206786365009. |
Updated by mfriedrich on 2014-09-16 09:19:50 00:00
|
This issue has been migrated from Redmine: https://dev.icinga.com/issues/6646
Created by tgelf on 2014-07-04 15:43:24 00:00
Assignee: gbeutner
Status: Resolved (closed on 2014-07-23 08:30:05 00:00)
Target Version: 2.0.2
Last Update: 2014-09-16 09:19:50 00:00 (in Redmine)
The icinga_objects table has to show NULL for name2 if there is no such property, not an empty string. The column is 'DEFAULT NULL', "" is just wrong here. I guess a lot of people have built queries based on this, I definitively did. And as far as I remember also the default Icinga Jasper reports do.
Cheers,
Thomas
Changesets
2014-07-22 08:29:09 00:00 by gbeutner f86068d
2014-07-23 08:25:26 00:00 by gbeutner 1a361d8
2014-07-23 08:48:33 00:00 by (unknown) 88d1081
The text was updated successfully, but these errors were encountered: