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

innodb web migration tool should give innodb status

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.3
    • Component/s: Database SQL/XMLDB
    • Labels:
      None
    • Environment:
      Moodle installed on LAMP server
    • Database:
      MySQL
    • Testing Instructions:
      Hide
      1. Setup moodle with mysql DB
      2. convert some tables back to myisam (easiest way is to use the admin/cli/mysql_engine.php CLI tool - php admin/cli/mysql_engine.php --engine=myisam )
      3. Visit admin/tool/innodb/and run migration
      4. Visit admin/tool/innodb/ again and run migration, it should tell you it was already converted
      Show
      Setup moodle with mysql DB convert some tables back to myisam (easiest way is to use the admin/cli/mysql_engine.php CLI tool - php admin/cli/mysql_engine.php --engine=myisam ) Visit admin/tool/innodb/and run migration Visit admin/tool/innodb/ again and run migration, it should tell you it was already converted
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w20_MDL-25197_m23_innodb

      Description

      If I go to http://MOODLE-SERVER/admin/innodb.php should give innodb status and click yes to "Are you sure you want convert all your tables to the InnoDB format?" and then go back to that URL after the conversion is done, all I see is the same questions. If removing the question after it was run is not easily possible or not desired, I think seeing a message that said something like:

      The conversion to InnoDB was last run on XYZ date. You do not need to reconvert.

      or something similar to that would be helpful.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            felicia Felicia Berryman added a comment -

            My first description doesn't make much sense... See again below:

            If I go to http://MOODLE-SERVER/admin/innodb.php and click yes to:
            "Are you sure you want convert all your tables to the InnoDB format?"
            and then later revisit http://MOODLE-SERVER/admin/innodb.php after the conversion is done, all I see is the same question asking me if I want to convert.

            I think seeing a message that said something like:
            "The conversion to InnoDB was last run on XYZ date. You do not need to reconvert."
            or something similar to that would be helpful.

            Show
            felicia Felicia Berryman added a comment - My first description doesn't make much sense... See again below: If I go to http://MOODLE-SERVER/admin/innodb.php and click yes to: "Are you sure you want convert all your tables to the InnoDB format?" and then later revisit http://MOODLE-SERVER/admin/innodb.php after the conversion is done, all I see is the same question asking me if I want to convert. I think seeing a message that said something like: "The conversion to InnoDB was last run on XYZ date. You do not need to reconvert." or something similar to that would be helpful.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Assigning to Petr

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Assigning to Petr
            Hide
            skodak Petr Skoda added a comment -

            Thanks for the report, I have added detection of existing InnoDB tables.

            Petr

            Show
            skodak Petr Skoda added a comment - Thanks for the report, I have added detection of existing InnoDB tables. Petr
            Hide
            poltawski Dan Poltawski added a comment -

            I've integrated this, thanks

            Show
            poltawski Dan Poltawski added a comment - I've integrated this, thanks
            Hide
            poltawski Dan Poltawski added a comment -

            THis works, but then I discovered.. MDL-33001

            Show
            poltawski Dan Poltawski added a comment - THis works, but then I discovered.. MDL-33001
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            U P S T R E A M I Z E D !

            Many thanks for the hard work, closing this as fixed.

            Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - U P S T R E A M I Z E D ! Many thanks for the hard work, closing this as fixed. Ciao

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  25/Jun/12