← Back to team overview

maria-developers team mailing list archive

Re: b1a4d1e4937: MDEV-16973 Application-time periods: DELETE

 

Ok, thanks.

On Fri, Feb 8, 2019 at 10:36 PM Sergei Golubchik <serg@xxxxxxxxxxx> wrote:

> Hi, Nikita!
>
> On Feb 08, Nikita Malyavin wrote:
>
> > > > +# auto_increment field overflow
> > > > +create or replace table t (id tinyint auto_increment primary key,
> > > > +s date, e date, period for apptime(s,e));
> > > > +insert into t values(127, '1999-01-01', '2018-12-12');
> > > > +delete from t for portion of apptime from '2000-01-01' to
> '2018-01-01';
> > > > +ERROR 22003: Out of range value for column 'id' at row 1
> > >
> > > add select * from t here, please
> > >
> >
> > > > +# same for trigger case
> > > > +delete from t for portion of apptime from '2000-01-01' to
> '2018-01-01';
> > > > +ERROR 22003: Out of range value for column 'id' at row 1
> > >
> > > and here
> > > and select from the log_tbl too
> >
> > Ahhh, the results differ for Innodb and MyISAM because of transactions.
> >
> > Well, I came across with solution, but it may look cumbersome. I've put
> the
> > queries in while loop and cycle the engines.
> > Please take a look at it in delete.test. Maybe You'll want it to be in
> > different file, or have any better idea/suggestion.
>
> I see. Then just add it commented, like
>
>  #select * from t;
>  #select * from log_tbl order by id;
>
> When I'll merge it, I'll uncomment and will see for myself how they
> differ and if it's worth keeping them.
>
> Generally the usual solution is to add an .rdiff file, but I can do that
> myself, just want to see the difference in results first.
>
> Regards,
> Sergei
> Chief Architect MariaDB
> and security@xxxxxxxxxxx
>


-- 
Yours truly,
Nikita Malyavin

References