|From:||Yugo NAGATA <nagata(at)sraoss(dot)co(dot)jp>|
|To:||Thomas Munro <thomas(dot)munro(at)gmail(dot)com>|
|Cc:||Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>, Marina Polyakova <m(dot)polyakova(at)postgrespro(dot)ru>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Teodor Sigaev <teodor(at)sigaev(dot)ru>|
|Subject:||Re: [HACKERS] WIP Patch: Pgbench Serialization and deadlock errors|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
On Tue, 10 Mar 2020 09:48:23 +1300
Thomas Munro <thomas(dot)munro(at)gmail(dot)com> wrote:
> On Tue, Mar 10, 2020 at 8:43 AM Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> wrote:
> > >> Thank you very much! I'm going to send a new patch set until the end of
> > >> this week (I'm sorry I was very busy in the release of Postgres Pro
> > >> 11...).
> > >
> > > Is anyone interested in rebasing this, and summarising what needs to
> > > be done to get it in? It's arguably a bug or at least quite
> > > unfortunate that pgbench doesn't work with SERIALIZABLE, and I heard
> > > that a couple of forks already ship Marina's patch set.
I got interested in this and now looking into the patch and the past discussion.
If anyone other won't do it and there are no objection, I would like to rebase
this. Is that okay?
> > I'm a reviewer on this patch, that I find a good thing (tm), and which was
> > converging to a reasonable and simple enough addition, IMHO.
> > If I proceed in place of Marina, who is going to do the reviews?
> Hi Fabien,
> Cool. I'll definitely take it for a spin if you post a fresh patch
> set. Any place that we arbitrarily don't support SERIALIZABLE, I
> consider a bug, so I'd like to commit this if we can agree it's ready.
> It sounds like it's actually in pretty good shape.
Yugo NAGATA <nagata(at)sraoss(dot)co(dot)jp>
|Next Message||Kyotaro Horiguchi||2021-05-24 02:34:02||Re: Race condition in recovery?|
|Previous Message||Greg Nancarrow||2021-05-24 02:04:37||Re: Re: Parallel scan with SubTransGetTopmostTransaction assert coredump|