pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  

help / color / mirror / Atom feed

From: Simon Riggs <simon@2ndQuadrant.com>
To: Heikki Linnakangas <heikki.linnakangas@enterprisedb.com>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Subject: Re: Hot Standby on git
Date: Mon, 05 Oct 2009 12:50:55 +0100
Message-ID: <1254743455.4691.153.camel@ebony.2ndQuadrant> (raw)
In-Reply-To: <4ABF4522.6060601@enterprisedb.com>
References: <1253958559.4449.833.camel@ebony.2ndQuadrant>
	<4ABF4522.6060601@enterprisedb.com>


On Sun, 2009-09-27 at 13:57 +0300, Heikki Linnakangas wrote:
> Per Simon's request, for the benefit of the archive, here's all the
> changes I've done on the patch since he posted the initial version for
> review for this commitfest as incremental patches. This is extracted
> from my git repository at
> git://git.postgresql.org/git/users/heikki/postgres.git.

I'm working my way through these changes now. 1, 2, 15 and 16 applied.

We discussed briefly your change 
0011-Replace-per-proc-counters-of-loggable-locks-with-per.patch.

I don't see how that helps at all. The objective of lock counters was to
know if we can skip acquiring an LWlock on all lock partitions. This
change keeps the lock counters yet acquires the locks we were trying to
avoid. This change needs some justification since it is not a bug fix.

-- 
 Simon Riggs           www.2ndQuadrant.com

view thread (48+ messages)  latest in thread

Message-ID: <1254743455.4691.153.camel@ebony.2ndQuadrant>
Permalink:  ../1254743455.4691.153.camel@ebony.2ndQuadrant/
Also on:    postgresql.org/message-id/1254743455.4691.153.camel@ebony.2ndQuadrant

copy link · copy postgr.es

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-hackers@postgresql.org
  Cc: simon@2ndQuadrant.com, heikki.linnakangas@enterprisedb.com
  Subject: Re: Hot Standby on git
  In-Reply-To: <1254743455.4691.153.camel@ebony.2ndQuadrant>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox