Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-63521

Moodle installer checks for innodb_file_per_table when it probably shouldn't

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 3.4.5, 3.5.2
    • Fix Version/s: None
    • Component/s: Installation
    • Database:
      MySQL
    • Affected Branches:
      MOODLE_34_STABLE, MOODLE_35_STABLE

      Description

      While I understand the logic of checking for Barracuda and large prefixes during the installation, I'm not so convinced about checking for innodb_file_per_table during installation.

      For smaller installs, being set either way is not particularly problematic, since if not enabled,  tables may be dropped and the space not immediately reclaimed by the filesystem, the space is still usable by MySQL.

      The problem comes in for us with larger installations when using AWS and Azure, namely that replication of databases on these environments works much better on having one large tablespace that all tables are part of, and performance is improved as well in our experience.

      I'd argue that this is also a decision that systems administrators should be making rather than applications; it should be transparent to the application whether it's on or off, and trying to impose logic for one set of users in the application to the detriment of others may not be ideal.

      For now we've simply resorted to manually removing that test during installation but it really shouldn't be application-level.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                peter.spicer Peter Spicer
                Participants:
                Component watchers:
                Matteo Scaramuccia, Andrew Nicols, Michael Hawkins
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated: