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

[dev.icinga.com #10061] IDO: icinga_host/service_groups alias columns are TEXT columns #3355

Closed
icinga-migration opened this issue Sep 1, 2015 · 15 comments

Comments

Projects
None yet
1 participant
@icinga-migration
Copy link
Member

commented Sep 1, 2015

This issue has been migrated from Redmine: https://dev.icinga.com/issues/10061

Created by tgelf on 2015-09-01 12:18:56 +00:00

Assignee: mfriedrich
Status: Resolved (closed on 2016-07-07 16:50:06 +00:00)
Target Version: 2.5.0
Last Update: 2016-11-11 08:46:47 +00:00 (in Redmine)

Icinga Version: 2.4.0
Backport?: Not yet backported
Include in Changelog: 1

  • used in search queries
  • no way to index them (we should do so)
  • notable performance impact

Changesets

2016-07-07 16:48:09 +00:00 by mfriedrich ca42b46

DB IDO: Change alias column to varchar(255) in MySQL

fixes #10061

Relations:

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Sep 1, 2015

Updated by tgelf on 2015-09-01 13:32:42 +00:00

  • Relates set to 9767
@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Sep 1, 2015

Updated by mfriedrich on 2015-09-01 14:20:51 +00:00

  • Parent Id set to 10073
@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Sep 4, 2015

Updated by mfriedrich on 2015-09-04 09:20:37 +00:00

  • Status changed from New to Assigned
@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Jan 23, 2016

Updated by mfriedrich on 2016-01-23 11:47:34 +00:00

  • Status changed from Assigned to New
  • Assigned to deleted tgelf
@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Jun 17, 2016

Updated by elippmann on 2016-06-17 08:12:19 +00:00

  • Status changed from New to Assigned
  • Assigned to set to mfriedrich
@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Jun 17, 2016

Updated by mfriedrich on 2016-06-17 08:17:33 +00:00

  • Assigned to changed from mfriedrich to elippmann

-> feature branch with changes :)

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Jun 17, 2016

Updated by elippmann on 2016-06-17 08:46:28 +00:00

Thanks :)

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Jun 28, 2016

Updated by mfriedrich on 2016-06-28 07:58:51 +00:00

  • Assigned to changed from elippmann to mfriedrich
@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Jul 6, 2016

Updated by mfriedrich on 2016-07-06 10:07:53 +00:00

  • Target Version set to 2.5.0
@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Jul 7, 2016

Updated by mfriedrich on 2016-07-07 16:33:22 +00:00

  • Relates set to 6382
@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Jul 7, 2016

Updated by mfriedrich on 2016-07-07 16:33:41 +00:00

  • Relates set to 2181
@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Jul 7, 2016

Updated by mfriedrich on 2016-07-07 16:47:36 +00:00

Eric and I agreed on changing the alias column to varchar(255) again. Possible optimised indexes won't be applied for the now ongoing 2.5 freeze but discussed in a separate meeting for IDO changes.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Jul 7, 2016

Updated by mfriedrich on 2016-07-07 16:50:06 +00:00

  • Status changed from Assigned to Resolved
  • Done % changed from 0 to 100

Applied in changeset ca42b46.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Jul 7, 2016

Updated by tgelf on 2016-07-07 17:03:43 +00:00

What's the reason for setting '' instead of null as a default value?

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Nov 11, 2016

Updated by mfriedrich on 2016-11-11 08:46:47 +00:00

  • Parent Id deleted 10073
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.