From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
---|---|
To: | Andres Freund <andres(at)anarazel(dot)de> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, Tatsuo Ishii <ishii(at)sraoss(dot)co(dot)jp>, craig(at)2ndquadrant(dot)com, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: REPLICA IDENTITY FULL |
Date: | 2017-06-23 17:14:30 |
Message-ID: | 20170623171430.xv7jfy72vqe4f73p@alvherre.pgsql |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Andres Freund wrote:
> On 2017-06-23 13:05:21 -0400, Alvaro Herrera wrote:
> > Tom Lane wrote:
> > > Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> writes:
> > > > Any thoughts about keeping datumAsEqual() as a first check? I did some
> > > > light performance tests, but it was inconclusive.
> > >
> > > Seems like it would tend to be a win if, in fact, the values are
> > > usually equal. If they're usually not, then it's a loser. Do
> > > we have any feeling for which case is more common?
>
> Seems like a premature optimization to me - if you care about
> performance and do this frequently, you're not going to end up using
> FULL. If we want to performance optimize, it'd probably better to
> lookup candidate keys and use those if available.
I can get behind that argument.
> > who in their right minds would use floating point columns as part of
> > replica identity ...?
>
> Since this is FULL, it'll be all columns...
Yeah, I was thinking you shouldn't have floating point columns if you're
going to use FULL as identity. But you're tautologically right: doing
the wrong thing is likely not the right thing to do.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2017-06-23 17:26:58 | Re: Logical replication: stuck spinlock at ReplicationSlotRelease |
Previous Message | Andres Freund | 2017-06-23 17:10:22 | Re: REPLICA IDENTITY FULL |