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

Latest revision of bb-10.3-svoj (big column compressed) does not compile

Details

    • Bug
    • Status: Closed (View Workflow)
    • Blocker
    • Resolution: Fixed
    • N/A
    • N/A
    • Compiling
    • None

    Description

      http://buildbot.askmonty.org/buildbot/builders/kvm-bintar-trusty-amd64/builds/2168/steps/compile/logs/stdio

      In file included from /home/buildbot/buildbot/build/client/mysqlbinlog.cc:3115:0:
      /home/buildbot/buildbot/build/sql/rpl_utility.cc: In member function 'uint32 table_def::calc_field_size(uint, uchar*) const':
      /home/buildbot/buildbot/build/sql/rpl_utility.cc:299:8: error: 'MYSQL_TYPE_VARCHAR_COMPRESSED' was not declared in this scope
         case MYSQL_TYPE_VARCHAR_COMPRESSED:
              ^
      /home/buildbot/buildbot/build/sql/rpl_utility.cc:309:8: error: 'MYSQL_TYPE_BLOB_COMPRESSED' was not declared in this scope
         case MYSQL_TYPE_BLOB_COMPRESSED:
              ^
      /home/buildbot/buildbot/build/sql/rpl_utility.cc: In constructor 'table_def::table_def(unsigned char*, ulong, uchar*, int, uchar*, uint16)':
      /home/buildbot/buildbot/build/sql/rpl_utility.cc:1074:12: error: 'MYSQL_TYPE_BLOB_COMPRESSED' was not declared in this scope
             case MYSQL_TYPE_BLOB_COMPRESSED:
                  ^
      /home/buildbot/buildbot/build/sql/rpl_utility.cc:1105:12: error: 'MYSQL_TYPE_VARCHAR_COMPRESSED' was not declared in this scope
             case MYSQL_TYPE_VARCHAR_COMPRESSED:
                  ^
      

      Attachments

        Issue Links

          Activity

            I explained this in a comment to MDEV-13342. I don't want to patch libmariadb yet, before this task is finally approved.

            To make it work currently, libmariadb has to be patched:

            diff --git a/include/mariadb_com.h b/include/mariadb_com.h
            index cdf7db8..31550cb 100644
            --- a/include/mariadb_com.h
            +++ b/include/mariadb_com.h
            @@ -327,6 +327,8 @@ enum enum_field_types { MYSQL_TYPE_DECIMAL, MYSQL_TYPE_TINY,
                                     MYSQL_TYPE_TIMESTAMP2,
                                     MYSQL_TYPE_DATETIME2,
                                     MYSQL_TYPE_TIME2,
            +                        MYSQL_TYPE_BLOB_COMPRESSED= 140,
            +                        MYSQL_TYPE_VARCHAR_COMPRESSED= 141,
                                     /* --------------------------------------------- */
                                     MYSQL_TYPE_JSON=245,
                                     MYSQL_TYPE_NEWDECIMAL=246,
            

            svoj Sergey Vojtovich added a comment - I explained this in a comment to MDEV-13342 . I don't want to patch libmariadb yet, before this task is finally approved. To make it work currently, libmariadb has to be patched: diff --git a/include/mariadb_com.h b/include/mariadb_com.h index cdf7db8..31550cb 100644 --- a/include/mariadb_com.h +++ b/include/mariadb_com.h @@ -327,6 +327,8 @@ enum enum_field_types { MYSQL_TYPE_DECIMAL, MYSQL_TYPE_TINY, MYSQL_TYPE_TIMESTAMP2, MYSQL_TYPE_DATETIME2, MYSQL_TYPE_TIME2, + MYSQL_TYPE_BLOB_COMPRESSED= 140, + MYSQL_TYPE_VARCHAR_COMPRESSED= 141, /* --------------------------------------------- */ MYSQL_TYPE_JSON=245, MYSQL_TYPE_NEWDECIMAL=246,

            We can patch it locally for testing. The problem is, it doesn't build in buildbot at all, so it doesn't undergo regression tests. It's a catch 22, you can't merge it to the main tree until buildbot confirms it's in a decent shape, and buildbot can't confirm it's in a decent shape because the relevant page hasn't pushed to the submodule's tree.

            According to serg, you should be able to create a development branch for libmariadb and use it instead.

            elenst Elena Stepanova added a comment - We can patch it locally for testing. The problem is, it doesn't build in buildbot at all, so it doesn't undergo regression tests. It's a catch 22, you can't merge it to the main tree until buildbot confirms it's in a decent shape, and buildbot can't confirm it's in a decent shape because the relevant page hasn't pushed to the submodule's tree. According to serg , you should be able to create a development branch for libmariadb and use it instead.

            Fixed in bb-10.3-svoj.

            svoj Sergey Vojtovich added a comment - Fixed in bb-10.3-svoj.

            People

              svoj Sergey Vojtovich
              elenst Elena Stepanova
              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.