Hi, Aleksey! On Oct 22, Aleksey Midenkov wrote:

revision-id: 83614913d9a (mariadb-10.4.11-291-g83614913d9a) parent(s): efb1023b6f4 author: Aleksey Midenkov <midenok@gmail.com> committer: Sergei Golubchik <serg@mariadb.com> timestamp: 2020-10-22 16:44:13 +0200 message:

MDEV-16243 [Warning] "Versioned table ...: partition ... is full, add more HISTORY partitions" floods the error log

diff --git a/sql/partition_info.cc b/sql/partition_info.cc index 8989a918c0c..f287cfddcd2 100644 --- a/sql/partition_info.cc +++ b/sql/partition_info.cc @@ -879,9 +879,12 @@ void partition_info::vers_set_hist_part(THD *thd) } return; warn: - my_error(WARN_VERS_PART_FULL, MYF(ME_WARNING|ME_ERROR_LOG), - table->s->db.str, table->s->table_name.str, - vers_info->hist_part->partition_name); + if (vers_info->hist_part->warn_vers_part_full) + { + my_error(WARN_VERS_PART_FULL, MYF(ME_WARNING|ME_ERROR_LOG), + table->s->db.str, table->s->table_name.str, + vers_info->hist_part->partition_name); + }

would it make sense to put warn_vers_part_full into vers_info, instead of storing it per-partition?

}

Regards, Sergei VP of MariaDB Server Engineering and security@mariadb.org