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

ERROR 138 instead of warning, and potential error masking

Details

    Description

      INSTALL PLUGIN Spider SONAME 'ha_spider.so';
      		 
      CREATE TABLE t ENGINE=Spider COMMENT='WRAPPER "mysql"';

      Leads to, on base:

      11.3.0 126157061b4376496c034a809ea4943e863d1465 (Debug)

      		 
      11.3.0-dbg> INSTALL PLUGIN Spider SONAME 'ha_spider.so';
      		 
      Query OK, 0 rows affected, 1 warning (0.285 sec)
      		 
       
      		 
      11.3.0-dbg> CREATE TABLE ts1 ENGINE=Spider COMMENT='WRAPPER "mysql",TABLE "t1"';
      		 
      ERROR 1429 (HY000): Unable to connect to foreign data source: localhost

      Versus the patch:

      11.4.0 f93c20081a8a505ac502850ec02630f95673dfba (Optimized)

      		 
      11.4.0-opt> INSTALL PLUGIN Spider SONAME 'ha_spider.so';
      		 
      Query OK, 0 rows affected, 1 warning (0.284 sec)
      		 
       
      		 
      11.4.0-opt> CREATE TABLE ts1 ENGINE=Spider COMMENT='WRAPPER "mysql",TABLE "t1"';
      		 
      ERROR 138 (0A000): Spider table params in COMMENT or CONNECTION strings have been deprecated and will be removed in a future release. Please use table options instead.

      Two issues:
      1) It seems that other errors are masked in favor of ERROR 138
      2) The ERROR should have been a warning only

      Attachments

        Issue Links

          blocks

          Task - A task that needs to be done. MDEV-28861 Spider: Deprecate table options by COMMENT

          • Critical - Crashes, loss of data, severe memory leak.
          • Closed
          is caused by

          Task - A task that needs to be done. MDEV-28861 Spider: Deprecate table options by COMMENT

          • Critical - Crashes, loss of data, severe memory leak.
          • Closed

          Activity

            Transition Time In Source Status Execution Times
            Yuchen Pei made transition -
            Open Closed
            		2d 14h 56m 1
            Roel Van de Paar made transition -
            Closed Stalled
            		1d 45m 1
            Roel Van de Paar made transition -
            Stalled Open
            		4s 1
            Roel Van de Paar made transition -
            Confirmed Open
            		2s 1
            Yuchen Pei made transition -
            Open Confirmed
            		37d 4h 48m 2
            Yuchen Pei made transition -
            Confirmed In Review
            		1m 48s 1
            Alexey Botchkov made transition -
            In Review Stalled
            		16h 23m 1
            Yuchen Pei made transition -
            Stalled In Testing
            		1h 33m 1
            Roel Van de Paar made transition -
            In Testing Stalled
            		23h 4m 1
            Yuchen Pei made transition -
            Stalled Closed
            		4h 54m 1

            People

              ycp Yuchen Pei
              Roel Roel Van de Paar
              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.