Skip to content

CLOUDSTACK-9481: Convert MyISAM table to InnoDB for consistency#1670

Merged
asfgit merged 1 commit intoapache:4.9from
shapeblue:innodb_4_9
Sep 1, 2016
Merged

CLOUDSTACK-9481: Convert MyISAM table to InnoDB for consistency#1670
asfgit merged 1 commit intoapache:4.9from
shapeblue:innodb_4_9

Conversation

@abhinandanprateek
Copy link
Contributor

No description provided.

ALTER TABLE cloud.nic_ip_alias ENGINE=INNODB;
ALTER TABLE cloud.sslcerts ENGINE=INNODB;
ALTER TABLE cloud.op_lock ENGINE=INNODB;
ALTER TABLE cloud.op_nwgrp_work ENGINE=INNODB;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For op_lock and op_nwgrp_work tables engine is MEMORY, not sure of the reason. Was there any perf. issues addressed? These exists since the start.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, they have existed since 4.0. I checked git history, I think there is nothing specific to this table, they can be migrated to InnoDB. I'm running a local 4.9.0 CloudStack environment with these changes and it is working alright.

@yadvr
Copy link
Member

yadvr commented Sep 1, 2016

LGTM, tested with a local installation, running it for past 3 days.

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
@borisstoyanov
Copy link
Contributor

LGTM, based on review.

@yadvr
Copy link
Member

yadvr commented Sep 1, 2016

Thanks @borisstoyanov
Based on the reviews, db upgrade and local env tests, I'll merge this.

@asfgit asfgit merged commit 180406e into apache:4.9 Sep 1, 2016
asfgit pushed a commit that referenced this pull request Sep 1, 2016
CLOUDSTACK-9481: Convert MyISAM table to InnoDB for consistency

* pr/1670:
  CLOUDSTACK-9481: Convert MyISAM table to InnoDB for consistency

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
lucas-a-martins pushed a commit to scclouds/cloudstack that referenced this pull request Jan 29, 2024
…r-level' into '4.18.0.0-scclouds'

Atualizar configuração `vpc.max.networks` e colocá-la à nível de _cluster_

Closes apache#1670

See merge request scclouds/scclouds!699
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.

5 participants