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

AliSQL: [Feature] Issue#26 ADD SECONDARY CLUSTERING KEY SUPPORT

Details

    Description

      Description:
      ------------
      1. Add support for CLUSTERING key type that is a clustering
      secondary key
       
      2. Add secondary clustering key support to selected places
      in the query optimizer
       
      ported from percona-server:
      http://bazaar.launchpad.net/~laurynas-biveinis/percona-server/tokudb-multiple-clust-keys/revision/568
      http://bazaar.launchpad.net/~laurynas-biveinis/percona-server/tokudb-clustering-query-opt/revision/5695
      

      https://github.com/alibaba/AliSQL/commit/cf5e15eade48b67a82d413cb3b4289bbfefe25cf

      Attachments

        Activity

          plinux, I guess this was needed for TokuDB support. I believe MariaDB has this (somewhat differently) implemented. Is there anything of interest in this patch?

          svoj Sergey Vojtovich added a comment - plinux , I guess this was needed for TokuDB support. I believe MariaDB has this (somewhat differently) implemented. Is there anything of interest in this patch?
          plinux Lixun Peng added a comment -

          svoj I can the parts of this patch's code exist in MariaDB, but not all (maybe MariaDB implement by different way), could you please show me the commit id about how MariaDB implemenet this patch's feature?

          plinux Lixun Peng added a comment - svoj I can the parts of this patch's code exist in MariaDB, but not all (maybe MariaDB implement by different way), could you please show me the commit id about how MariaDB implemenet this patch's feature?

          I believe it was added in this patch: https://github.com/MariaDB/server/commit/3631146442bc09e5129334774b4bcdf2470a4b27

          Please note that in MariaDB you don't need to edit a whole lot of files to extend SQL syntax with new index option. E.g. TokuDB CLUSTERING index option is implemented this way: https://github.com/MariaDB/server/blob/10.2/storage/tokudb/tokudb_sysvars.cc#L1133

          svoj Sergey Vojtovich added a comment - I believe it was added in this patch: https://github.com/MariaDB/server/commit/3631146442bc09e5129334774b4bcdf2470a4b27 Please note that in MariaDB you don't need to edit a whole lot of files to extend SQL syntax with new index option. E.g. TokuDB CLUSTERING index option is implemented this way: https://github.com/MariaDB/server/blob/10.2/storage/tokudb/tokudb_sysvars.cc#L1133
          plinux Lixun Peng added a comment -

          svoj As we checked, MariaDB has the same feature, but different implement, so you needn't to do anything for this issue.

          plinux Lixun Peng added a comment - svoj As we checked, MariaDB has the same feature, but different implement, so you needn't to do anything for this issue.

          Closing per plinux comment.

          svoj Sergey Vojtovich added a comment - Closing per plinux comment.

          People

            plinux Lixun Peng
            svoj Sergey Vojtovich
            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.