Details

    Description

      mysqldump --header --tab wraps the dumping SELECT in UNION ALL + ORDER BY. Which means a filesort (try EXPLAIN), that would make mysqldump notably slower on large tables. This is completely unnecessary, because the ordering of table rows is unimportant.

      Attachments

        Issue Links

          Activity

            rmfalves Rui Alves added a comment -

            The ORDER BY that I implemented on the MDEV-22200 has two purposes:

            1. By using a temporary dummy binary sorting variable (0 when at the header row, 1 otherwise) it enforces the header row on the top and prevents it from getting messed up somewhere in the middle of the data rows.

            2. The ordering of data rows is necessary to enforce compability with the --order-by-primary clause.

            rmfalves Rui Alves added a comment - The ORDER BY that I implemented on the MDEV-22200 has two purposes: 1. By using a temporary dummy binary sorting variable (0 when at the header row, 1 otherwise) it enforces the header row on the top and prevents it from getting messed up somewhere in the middle of the data rows. 2. The ordering of data rows is necessary to enforce compability with the --order-by-primary clause.
            danblack Daniel Black added a comment -

            MDEV-12879 as option.

            danblack Daniel Black added a comment - MDEV-12879 as option.
            serg Sergei Golubchik added a comment - https://github.com/MariaDB/server/pull/2311/commits/b4d1ff1d3b262a35c1d83acf893544de9ccdaad0 is ok to push

            People

              danblack Daniel Black
              serg Sergei Golubchik
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Git Integration

                  Error rendering 'com.xiplink.jira.git.jira_git_plugin:git-issue-webpanel'. Please contact your Jira administrators.