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

Spider: remove unnecessary #ifdef and dead code in #ifdef

    XMLWordPrintable

    Details

      Description

      Remove unnecessary #ifdefs and dead code, in Spider, which is protected by #ifdef.

      Some time ago, the Spider was a separate project, and Kentoku tried to make it compile on many MySQL and MariaDB versions without changes. So, there are many #ifdef macro in the Spider codebase, which were to adjust Spider for changes in API over different server versions. For example, see ITEM_FUNC_CASE_PARAMS_ARE_PUBLIC. Now, the Spider is part of the source tree, and thus we can remove dead code in #ifdef.

      Use unifdef as far as possible.

        Attachments

          Issue Links

          1.
          Spider: remove #if defined(MARIADB_BASE_VERSION) && MYSQL_VERSION_ID >= ${VERSION} Technical task Closed Nayuta Yanagisawa
          2.
          Spider: remove #if MYSQL_VERSION_ID < ${VERSION} Technical task Closed Nayuta Yanagisawa
          3.
          Spider: remove #ifdef WITHOUT_SPIDER_BG_SEARCH Technical task Closed Nayuta Yanagisawa
          4.
          Spider: remove #ifdef HA_CAN_BULK_ACCESS Technical task Closed Nayuta Yanagisawa
          5.
          Spider: remove #ifdef HANDLER_HAS_DIRECT_AGGREGATE Technical task Closed Nayuta Yanagisawa
          6.
          Spider: remove #ifdef HA_MRR_USE_DEFAULT_IMPL Technical task Closed Nayuta Yanagisawa
          7.
          Spider: remove #ifdef SPIDER_HAS_HASH_VALUE_TYPE Technical task Closed Nayuta Yanagisawa
          8.
          Spider: remove #ifdef HANDLER_HAS_DIRECT_UPDATE_ROWS Technical task Closed Nayuta Yanagisawa
          9.
          Spider: remove #ifdef HASH_UPDATE_WITH_HASH_VALUE Technical task Closed Nayuta Yanagisawa
          10.
          Spider: remove #ifdef SPIDER_HAS_GROUP_BY_HANDLER Technical task Closed Nayuta Yanagisawa
          11.
          Spider: remove #ifdef HA_HAS_CHECKSUM_EXTENDED Technical task Closed Nayuta Yanagisawa
          12.
          Spider: remove #ifdef MARIADB_BASE_VERSION Technical task Closed Nayuta Yanagisawa
          13.
          Spider: remove #ifdef SPIDER_HAS_DISCOVER_TABLE_STRUCTURE Technical task Closed Nayuta Yanagisawa
          14.
          Spider: remove #ifdef SPIDER_HANDLER_SUPPORT_MULTIPLE_KEY_PARTS Technical task Closed Nayuta Yanagisawa
          15.
          Spider: remove #if defined(PARTITION_HAS_GET_CHILD_HANDLERS) Technical task Closed Nayuta Yanagisawa
          16.
          Spider: remove #ifdef HANDLER_HAS_DIRECT_UPDATE_ROWS_WITH_HS Technical task Closed Nayuta Yanagisawa
          17.
          Spider: remove #ifdef SPIDER_HANDLER_START_BULK_INSERT_HAS_FLAGS Technical task Closed Nayuta Yanagisawa
          18.
          Spider: remove #ifdef SPIDER_SUPPORT_CREATE_OR_REPLACE_TABLE Technical task Closed Nayuta Yanagisawa
          19.
          Spider: remove #ifdef SPIDER_USE_CONST_ITEM_FOR_STRING_INT_REAL_DECIMAL_DATE_ITEM Technical task Closed Nayuta Yanagisawa
          20.
          Spider: remove #ifdef SPIDER_SQL_CACHE_IS_IN_LEX Technical task Closed Nayuta Yanagisawa
          21.
          Spider: remove #ifdef SPIDER_LIKE_FUNC_HAS_GET_NEGATED Technical task Closed Nayuta Yanagisawa
          22.
          Spider: remove #ifdef SPIDER_I_S_USE_SHOW_FOR_COLUMN Technical task Closed Nayuta Yanagisawa
          23.
          Spider: remove #ifdef SPIDER_USE_CONST_ITEM_FOR_STRING_INT_REAL_DECIMAL_DATE_ITEM Technical task Closed Nayuta Yanagisawa
          24.
          Spider: remove #ifdef SPIDER_MDEV_16246 Technical task Closed Nayuta Yanagisawa
          25.
          Spider: remove #ifdef HANDLER_HAS_NEED_INFO_FOR_AUTO_INC Technical task Closed Nayuta Yanagisawa
          26.
          Spider: remove #ifdef SPIDER_HAS_MY_CHARLEN Technical task Closed Nayuta Yanagisawa
          27.
          Spider: remove #ifdef SPIDER_use_LEX_CSTRING_for_KEY_Field_name Technical task Closed Nayuta Yanagisawa
          28.
          Spider: remove #ifdef SPIDER_ITEM_STRING_WITHOUT_SET_STR_WITH_COPY_AND_THDPTR Technical task Closed Nayuta Yanagisawa
          29.
          Spider: remove #ifdef SPIDER_ITEM_STRING_WITHOUT_SET_STR_WITH_COPY Technical task Closed Nayuta Yanagisawa
          30.
          Spider: remove #ifdef SPIDER_use_LEX_CSTRING_for_Field_blob_constructor Technical task Open Nayuta Yanagisawa
          31.
          Spider: remove #ifdef SPIDER_HAS_EXPR_CACHE_ITEM Technical task Closed Nayuta Yanagisawa
          32.
          Spider: remove #ifdef SPIDER_ITEM_HAS_CMP_TYPE Technical task Closed Nayuta Yanagisawa
          33.
          Spider: remove #ifdef ITEM_FUNC_CASE_PARAMS_ARE_PUBLIC Technical task Open Nayuta Yanagisawa
          34.
          Spider: remove #ifdef SPIDER_XID_USES_xid_cache_iterate Technical task Closed Nayuta Yanagisawa
          35.
          Spider: remove #ifdef SPIDER_FIELD_FIELDPTR_REQUIRES_THDPTR Technical task Open Nayuta Yanagisawa
          36.
          Spider: remove #ifdef SPIDER_Item_args_arg_count_IS_PROTECTED Technical task Open Nayuta Yanagisawa
          37.
          Spider: remove #ifdef SPIDER_NET_HAS_THD Technical task Open Nayuta Yanagisawa
          38.
          Spider: remove #ifdef HA_EXTRA_HAS_STARTING_ORDERED_INDEX_SCAN Technical task Open Nayuta Yanagisawa
          39.
          Spider: remove #ifdef HANDLER_HAS_CAN_USE_FOR_AUTO_INC_INIT Technical task Open Nayuta Yanagisawa
          40.
          Spider: remove #ifdef SPIDER_UPDATE_ROW_HAS_CONST_NEW_DATA Technical task Open Nayuta Yanagisawa
          41.
          Spider: remove #ifdef ITEM_FUNC_TIMESTAMPDIFF_ARE_PUBLIC Technical task Open Nayuta Yanagisawa

            Activity

              People

              Assignee:
              nayuta-yanagisawa Nayuta Yanagisawa
              Reporter:
              nayuta-yanagisawa Nayuta Yanagisawa
              Votes:
              0 Vote for this issue
              Watchers:
              4 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.