have_innodb variable is deprecated and removed in MySQL 5.6.

[erenkarayigit]

SHOW ENGINES used to check InnoDB.

see http://dev.mysql.com/doc/refman/5.1/en/server-system-variables.html#sysvar_have_innodb

[jeffmoser]

This is a critical issue that effectively prevents installation to anyone using MySQL 5.6 and aren't able to fix the issue on their own. MySQL 5.6 was released in 2011. erenkarayigit committed a working change six months ago, but nobody has even been assigned to review it.

I don't mean for this to sound like an angry rant, but I'm wondering:

• is anyone assigned to the CE is monitoring the repo?
• what unaddressed issues might be lurking after installation?
• is there any type of regular release schedule for the CE?

[verklov]

Hello erenkarayigit,
Sorry for the delay with the response.
Thank you for your contribution! Our team will review your pull request and respond as soon as our analysis is complete.

[verklov]

Hello erenkarayigit,
We have this fix already in the code on Magento 2:
$variables = $this->_getConnection()->fetchPairs('SHOW ENGINES');
return isset($variables['InnoDB']) && ($variables['InnoDB'] == 'DEFAULT' || $variables['InnoDB'] == 'YES');

Thank you for your contribution and participation in Magento product development!