Uploaded image for project: 'MariaDB Server'
  1. MariaDB Server
  2. MDEV-5265

have_innodb should be restored in MariaDB 10

    Details

    • Type: Task
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Fix Version/s: 10.0
    • Component/s: None
    • Labels:
      None

      Description

      I think the have_innodb variable should be restored in MariaDB 10 as there are some PHP applications (tested with Magento) that check for the existance of INNODB storage engine with the contents of that variable.

      In their database connector I had to change:

      $variables  = $this->_getConnection()
                  ->fetchPairs('SHOW VARIABLES');
              return (!isset($variables['have_innodb']) || $variables['have_innodb'] != 'YES') ? false : true;
      

      To

      $engines  = $this->_getConnection()
                  ->fetchPairs('SHOW ENGINES');
      		return (!isset($engines['InnoDB']) || $engines['InnoDB'] != 'YES') ? false : true;
      

      And managed it to pass the check.

      Still I'd like that the have_innodb removal is reverted in order to restore compatibility with other software.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            serg Sergei Golubchik added a comment -

            without modifying the server, this can be done with a special "have" daemon plugin that provides missing "have_xxx" variables for whoever needs them.

            Show
            serg Sergei Golubchik added a comment - without modifying the server, this can be done with a special "have" daemon plugin that provides missing "have_xxx" variables for whoever needs them.
            Hide
            Jorge Silva Jorge Manuel Silva added a comment -

            And where can I download that?

            Show
            Jorge Silva Jorge Manuel Silva added a comment - And where can I download that?
            Hide
            serg Sergei Golubchik added a comment -

            You misunderstood. I didn't say it's done, I only provided a suggestion how it can be fixed without adding obsolete features back to the server.

            Show
            serg Sergei Golubchik added a comment - You misunderstood. I didn't say it's done, I only provided a suggestion how it can be fixed without adding obsolete features back to the server.
            Hide
            elenst Elena Stepanova added a comment -

            It seems to be a good task to contribute code (as a plugin) if somebody wants to do some community work.

            Show
            elenst Elena Stepanova added a comment - It seems to be a good task to contribute code (as a plugin) if somebody wants to do some community work.

              People

              • Assignee:
                Unassigned
                Reporter:
                Jorge Silva Jorge Manuel Silva
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated: