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

Unfortunate error message WARN_VERS_PART_FULL (partition <name> is full) when rotation time for the last interval passed

Details

    Description

      create or replace table t1 (a int) with system versioning 
      partition by system_time interval 1 second (
        partition p0 history, 
        partition p1 history, 
        partition pc current
      );
      -- wait for 3 seconds
      insert into t1 values (1),(2),(3);
      show warnings;
      

      10.3 9f84451d87

      +---------+------+----------------------------------------------------------------------------------+
      | Level   | Code | Message                                                                          |
      +---------+------+----------------------------------------------------------------------------------+
      | Warning | 4114 | Versioned table `test`.`t1`: partition `p1` is full, add more HISTORY partitions |
      +---------+------+----------------------------------------------------------------------------------+
      1 row in set (0.00 sec)
      

      The warning is not wrong (apart being thrown at a strange time, when no historical rows are created; but for that we have a separate JIRA item). However, the text is confusing – the partition is not full, it's just expired.

      Attachments

        Activity

          I think no warning should be here. All the data is just added into pn, and p1, p2 are not even touched

          nikitamalyavin Nikita Malyavin added a comment - I think no warning should be here. All the data is just added into pn, and p1, p2 are not even touched
          danblack Daniel Black added a comment -

          nikitamalyavin there's a PR https://github.com/MariaDB/server/pull/825 waiting that seems to agree with your previous comment. Do you think its correct? Can you merge it?

          danblack Daniel Black added a comment - nikitamalyavin there's a PR https://github.com/MariaDB/server/pull/825 waiting that seems to agree with your previous comment. Do you think its correct? Can you merge it?

          danblack Thanks! Merged. And added the test you asked

          nikitamalyavin Nikita Malyavin added a comment - danblack Thanks! Merged. And added the test you asked

          People

            serg Sergei Golubchik
            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.