1. 03 Jan, 2023 1 commit
    • Daniel Black's avatar
      MDEV-30275: mariadb names rather than mysql names should be used · c4938eaf
      Daniel Black authored
      * mariadb-service-convert to use mariadbd-safe
      * galera_recovery to use mariadbd
      * mtr - wsrep use mariadb executables
      * debian/mariadb-server.mariadb.init use mariadbd-safe
      * debian/tests/smoke uses mariadb instead of mysql as client.
      
      Co-Author: Daniel Black <daniel@mariadb.org>
      c4938eaf
  2. 02 Jan, 2023 1 commit
  3. 19 Dec, 2022 1 commit
  4. 18 Dec, 2022 1 commit
  5. 16 Dec, 2022 1 commit
  6. 15 Dec, 2022 3 commits
    • Monty's avatar
      MDEV-30118 exception in ha_maria::extra · d0cd4949
      Monty authored
      I have not been able to repeat the problem, but the stack trace indicates
      that ha_maria::extra() is called with a null file pointer.
      
      This indicates the table has either never been opened or opened and closed,
      with file pointer set to NULL, but ha_maria::extra() is still called.
      
      In JOIN::partial_cleanup() we are only checking of table->is_created(),
      which will fail if table was created and later closed.
      
      Fixed by clearing table->created if table is dropped.
      
      I added an assert to is_created() to catch the case that the create
      flag does not match 'file'.
      d0cd4949
    • Marko Mäkelä's avatar
      MDEV-30227 [ERROR] [FATAL] InnoDB: fdatasync() returned 9 · 92ff7bb6
      Marko Mäkelä authored
      fil_space_t::flush<false>(): If the CLOSING flag is set,
      the file may already have been closed, resulting in EBADF
      being returned by fdatasync(). In any case, the
      thread that had set the flag should take care of invoking
      os_file_flush_func().
      
      The crash occurred during the execution of FLUSH TABLES...FOR EXPORT.
      
      Tested by: Matthias Leich
      92ff7bb6
    • Daniel Black's avatar
      03fee585
  7. 14 Dec, 2022 6 commits
  8. 13 Dec, 2022 5 commits
  9. 12 Dec, 2022 3 commits
  10. 10 Dec, 2022 1 commit
    • Julius Goryavsky's avatar
      MDEV-29814: galera_var_notify_ssl_ipv6 causes testing system to hang · a4914008
      Julius Goryavsky authored
      This commit fixes the test system hanging due to
      the galera_var_notify_ssl_ipv6 test and also brings
      the wsrep_notify[_ssl].sh files in line with each other
      between the user template and the mtr suite.
      
      Quotes are also added here to avoid problems if the
      user specifies the value of one of the variables at the
      beginning of the file containing shell-specific characters,
      for example, if the password or username specified in the
      PSWD and USER variables will contain the "$" character.
      
      Also fixed an issue with automatic --ssl-verify-server-cert
      option substitution when the corresponding value is set
      by the user to "1" or "on".
      
      Also fixed some tests here to avoid joining one of the nodes
      to another cluster when the nodes are restarted from the mtr
      side, which can lead to random failures when testing with
      buildbot.
      a4914008
  11. 09 Dec, 2022 2 commits
    • Julius Goryavsky's avatar
      MDEV-29814: galera_var_notify_ssl_ipv6 causes testing system to hang · 8f309732
      Julius Goryavsky authored
      This commit fixes the test system hanging due to
      the galera_var_notify_ssl_ipv6 test and also brings
      the wsrep_notify[_ssl].sh files in line with each other
      between the user template and the mtr suite.
      
      Quotes are also added here to avoid problems if the
      user specifies the value of one of the variables at the
      beginning of the file containing shell-specific characters,
      for example, if the password or username specified in the
      PSWD and USER variables will contain the "$" character.
      
      Also fixed an issue with automatic --ssl-verify-server-cert
      option substitution when the corresponding value is set
      by the user to "1" or "on".
      
      Also fixed some tests here to avoid joining one of the nodes
      to another cluster when the nodes are restarted from the mtr
      side, which can lead to random failures when testing with
      buildbot.
      8f309732
    • Marko Mäkelä's avatar
      MDEV-29144 ER_TABLE_SCHEMA_MISMATCH or crash on DISCARD/IMPORT · 782b2a75
      Marko Mäkelä authored
      mysql_discard_or_import_tablespace(): On successful
      ALTER TABLE...DISCARD TABLESPACE, evict the table handle from the
      table definition cache, so that ha_innobase::close() will be invoked,
      like InnoDB expects to be the case. This will avoid an assertion failure
      ut_a(table->get_ref_count() == 0) during IMPORT TABLESPACE.
      
      ha_innobase::open(): Do not issue any ER_TABLESPACE_DISCARDED warning.
      Member functions for DML will do that.
      
      ha_innobase::truncate(), ha_innobase::check_if_supported_inplace_alter():
      Issue ER_TABLESPACE_DISCARDED warnings, to compensate for the removal of
      the warning in ha_innobase::open().
      
      row_quiesce_write_indexes(): Only write information about committed
      indexes. The ALTER TABLE t NOWAIT ADD INDEX(c) in the nondeterministic
      test case will most of the time fail due to a metadata lock (MDL) timeout
      and leave behind an uncommitted index.
      
      Reviewed by: Sergei Golubchik
      782b2a75
  12. 08 Dec, 2022 2 commits
    • Daniel Black's avatar
      MDEV-30150 ST_GeomFromGeoJSON, 'geometry' before 'type: feature' error · 8f3631d0
      Daniel Black authored
      The geometry type requires Type:"Feature" but the feature need
      not be first in the JSON structure.
      
      Adjust code to return an error if geometry isn't a JSON object,
      but continue parsing searching for Type: "Feature" to trigger
      the geometry parsing.
      
      Thanks Derick Magnusen for the bug report.
      8f3631d0
    • Tuukka Pasanen's avatar
      MDEV-28834: Add minimal support for Lintian version 2.115 and above · 85181653
      Tuukka Pasanen authored
      Convert minimal amount of Lintian overrides to make Lintian
      test pass also with Debian Sid latest Lintian 2.115 version.
      
      Old style overrides are kept so they can be used with
      older versions of Lintian.
      
      Introduce minimal Lintian overrides which are common
      from MariaDB version 10.5 up-to to 10.8.
      
      Overrides added files:
        * debian/mariadb-test-data.lintian-overrides
          - MariaDB installs some shared objects to test-suite directory and not in
            '/usr/lib' or similar. Share objects is pam_mariadb_mtr.so. Tags are
            arch-dependent-file-in-usr-share and
            arch-independent-package-contains-binary-or-object Lintia
       * debian/mariadb-test.lintian-overrides
         - MariaDB installs some some binaries to test-sute directory and
           in mariadb-test package they are my_safe_process and
           wsrep_check_version. Tags is
           arch-dependent-file-in-usr-share
       * debian/source/lintian-overrides
         - In source there is some source files missing which should be addressed
           sql/share/charsets/languages.html and
           and storage/rocksdb/rocksdb/docs/_includes/footer.html.
           Tags is source-is-missing
         - Add Lintian override for missing:
           storage/columnstore/columnstore/utils/jemalloc/libjemalloc.so.2
         - Add Lintian override for substvar external resources:
           ${source:Version} libmariadb-dev -> libmysqlclient-dev [debian/control:66]
           ${source:Version} libmariadb-dev -> libmysqld-dev [debian/control:66]
           ${source:Version} libmariadbd-dev -> libmariadbclient-dev [debian/control:216]
      85181653
  13. 07 Dec, 2022 3 commits
  14. 05 Dec, 2022 2 commits
    • Anel Husakovic's avatar
      dgcov: parsing without dot to get specific version · a59dffb0
      Anel Husakovic authored
      - Regression introduced 7baf24a0 for multidigit gcc dump.
      There is no dot in `dumpversion`.
      ```
      $ gcc -dumpversion
      10
      ```
      Otherwise it will fail and not produce the output
      ```
      Running dgcov
      Name "IO::Uncompress::Gunzip::GunzipError" used only once: possible typo at ./dgcov.pl line 197.
      Cannot parse gcc -dumpversion: 9
      ```
      
      - The warning `once` is always generated:
      ```
      Running dgcov
      Name "IO::Uncompress::Gunzip::GunzipError" used only once: possible typo at ./dgcov.pl line 197.
      <number>
      ```
      Suppresing the line `Name "IO::Uncompress::Gunzip::GunzipError" used only once: possible typo at ./dgcov.pl line 197.`
      with the patch.
      
      - Reviewed by: <>
      a59dffb0
    • Monty's avatar
      Fixed a crash during automatic zerofill of moved Aria table · e748f5cc
      Monty authored
      This could happen if one did a DML with a moved table that one had done
      an external zerofill on.
      The crash happend because a message that was supposed to be sent to
      a repair report was instead sent to the result, which caused an ASSERT
      e748f5cc
  15. 03 Dec, 2022 1 commit
  16. 02 Dec, 2022 7 commits