Just nit-picking, but MyISAM allows replication as well. I'm +1 to the change.
-- Erik On Wed, Aug 31, 2016 at 5:37 AM, Abhinandan Prateek < abhinandan.prat...@shapeblue.com> wrote: > Thanks, Koushik. > > Have created a PR to make the engine type consistent for all tables. > InnoDB allows for replication, that is all the more reason that we change > the engine type of these few tables to InnoDB. > > Regards, > -abhi > > PR: https://github.com/apache/cloudstack/pull/1667 > > > From: Koushik Das <koushik....@accelerite.com<mailto: > koushik....@accelerite.com>> > Date: Tuesday, 30 August 2016 at 5:25 PM > To: Abhinandan Prateek <abhinandan.prat...@shapeblue.com<mailto: > abhinandan.prat...@shapeblue.com>>, "dev@cloudstack.apache.org<mailto: > dev@cloudstack.apache.org>" <dev@cloudstack.apache.org<mailto: > dev@cloudstack.apache.org>> > Cc: Rajani Karuturi <raj...@apache.org<mailto:raj...@apache.org>>, Raja > Pullela <raja.pull...@accelerite.com<mailto:raja.pull...@accelerite.com>> > Subject: Re: Few tables in cloudstack schema use MyISAM engine > > I cannot think of any. Most probably in all cases the engine was not > explicitly specified and the default got used. As per below default was > myisam before mysql 5.5.5. > https://dev.mysql.com/doc/refman/5.5/en/storage-engine-setting.html > > Thanks, > Koushik > From: Abhinandan Prateek <abhinandan.prat...@shapeblue.com<mailto: > abhinandan.prat...@shapeblue.com>> > Date: Tuesday, 30 August 2016 at 4:23 PM > To: "dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org>" < > dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org>> > Cc: Rajani Karuturi <raj...@apache.org<mailto:raj...@apache.org>>, Raja > Pullela <raja.pull...@accelerite.com<mailto:raja.pull...@accelerite.com>>, > Koushik Das <koushik....@accelerite.com<mailto:koushik....@accelerite.com > >> > Subject: Few tables in cloudstack schema use MyISAM engine > > Hi, > > Was there any specific reason that some tables in cloudstack schema use > MyISAM engine. > I know of “ quota_account” that I created and put in the diff table type > unknowingly. I assume such is the case with other such tables – > load_balancer_cert_map, monitoring_services, nic_ip_alias, sslcerts. > > Filed a ticket here to get this fixed or reason for not fixing it captured. > > Regards, > -abhi > > abhinandan.prat...@shapeblue.com<mailto:abhinandan.prat...@shapeblue.com> > www.shapeblue.com > @shapeblue > > > > > DISCLAIMER ========== This e-mail may contain privileged and confidential > information which is the property of Accelerite, a Persistent Systems > business. It is intended only for the use of the individual or entity to > which it is addressed. If you are not the intended recipient, you are not > authorized to read, retain, copy, print, distribute or use this message. If > you have received this communication in error, please notify the sender and > delete all copies of this message. Accelerite, a Persistent Systems > business does not accept any liability for virus infected mails. > > abhinandan.prat...@shapeblue.com > www.shapeblue.com > 53 Chandos Place, Covent Garden, London WC2N 4HSUK > @shapeblue > > > >