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

Implement MySQL 8.0 native functions: format_bytes(), format_pico_time() and ps_thread_id()

Details

    Description

      While this functions can be implemented as user SQL functions, they have become native on MySQL 8.0.16. They are simple and independent enough that backport them should be easy.
      Implementing them on MariaDB would allow:

      • Better compatibility with sys/performance_schema
      • Nice functions to have natively in any case for formatting reasons or even as a virtual column
      • Better MySQL compatibility, and avoiding user pain when migrating from MySQL from 8.0.16+
      • Better performance by being C rather than user functions/SQL

      https://mysql.wisborg.dk/2019/05/28/performance-schema-functions/

      MySQL implementation

      MariaDB implementation

      • MariaDB doesn't have any native functions
      • MariaDB has sys schema function format_bytes, as stated in sysschema.format_bytes documentation.
      • This change is introduced from 10.6.0 with commits and author 

        commit aa2ff62082c4eb09b1d505af0d0327278c9f7d08
        		 
        Author: Vladislav Vaintroub <wlad@mariadb.com>
        		 
        Date:   Wed Mar 3 10:30:29 2021 +0100
        		 
         
        		 
            MDEV-9077 Use sys schema in bootstrapping, incl. mtr

        and second commit

        commit 4bac804c90c6f8d101ef32aef39813c3d13283ff
        		 
        Author: Vladislav Vaintroub <wlad@mariadb.com>
        		 
        Date:   Wed Mar 17 17:57:11 2021 +0100
        		 
         
        		 
            MDEV-9077 add sys_schema from https://github.com/mysql/mysql-sys

      • Tasks:
        The task can be divided into 3 steps by backporting the patch from MySQL if possible in single commit and if not in separate commits:
      • Create native function format_pico_time - task done as contribution with PR 2558 on 11.0, commit d9808f79
      • Create native function ps_thread_id
      • Create native function ps_current_thread_id
      • Create native function for format_bytes

      Attachments

        Issue Links

          causes

          Bug - A problem which impairs or prevents the functions of the product. MDEV-36020 ASAN errors in Binary_string::replace upon using format_pico_time in view

          • Major - Major loss of function.
          • Open
          includes

          New Feature - A new feature of the product, which has yet to be developed. MDEV-31736 Create native function for format_bytes

          • Major - Major loss of function.
          • Closed
          is part of

          Task - A task that needs to be done. MDEV-28906 MySQL 8.0 desired compatibility

          • Major - Major loss of function.
          • Open
          relates to

          Bug - A problem which impairs or prevents the functions of the product. MDEV-33703 Negative numbers and differnce of behaviour between sys.format_time() and FORMAT_PICO_TIME()

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Open

          Activity

            Ascending order - Click to sort in descending order
            Abdelfatah Abdelrahman added a comment -

            Is any one working on "Create native function for format_bytes" ? I would like to start working on it.
            I found out that "format_bytes" is deprecated according to this https://dev.mysql.com/doc/refman/8.0/en/sys-format-bytes.html , and there's a new version https://dev.mysql.com/doc/refman/8.0/en/performance-schema-functions.html#function_format-bytes , which one should I work on ? the difference is minimal anyways and it would be easy to implement.

            Abdelfatah Abdelrahman added a comment - Is any one working on "Create native function for format_bytes" ? I would like to start working on it. I found out that "format_bytes" is deprecated according to this https://dev.mysql.com/doc/refman/8.0/en/sys-format-bytes.html , and there's a new version https://dev.mysql.com/doc/refman/8.0/en/performance-schema-functions.html#function_format-bytes , which one should I work on ? the difference is minimal anyways and it would be easy to implement.
            jcrespo Jaime Crespo added a comment -

            @Abdelrahman please note that what's deprecated is the sys stored procedure, precisely because they exist in MySQL as built-in functions (and why this was filed). The latter version is what's asked.

            I would ask Anel first before taking it, as they started working on one of the subtasks.

            jcrespo Jaime Crespo added a comment - @Abdelrahman please note that what's deprecated is the sys stored procedure, precisely because they exist in MySQL as built-in functions (and why this was filed). The latter version is what's asked. I would ask Anel first before taking it, as they started working on one of the subtasks.

            People

              Unassigned Unassigned
              jcrespo Jaime Crespo
              Votes:
              1 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:

                Git Integration

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