maria-developers team mailing list archive

[Sergei Golubchik <serg@xxxxxxxxxxx>]

Hi, Aleksey!

On Oct 22, Aleksey Midenkov wrote:
> revision-id: 83614913d9a (mariadb-10.4.11-291-g83614913d9a)
> parent(s): efb1023b6f4
> author: Aleksey Midenkov <midenok@xxxxxxxxx>
> committer: Sergei Golubchik <serg@xxxxxxxxxxx>
> 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@xxxxxxxxxxx