public inbox for [email protected]  

help / color / mirror / Atom feed

From: Tatsuo Ishii <[email protected]>
To: [email protected]
Cc: [email protected]
Subject: Re: Proposal: Recent mutated table tracking in memory
Date: Fri, 30 Jan 2026 17:09:50 +0900 (JST)
Message-ID: <[email protected]> (raw)
In-Reply-To: <CACeKOO36wVT2k5b9G2J4XYvjNQTYf-ARoh8Lp-jbh7Yo8Qo+-w@mail.gmail.com>
References: <[email protected]>
	<[email protected]>
	<CACeKOO36wVT2k5b9G2J4XYvjNQTYf-ARoh8Lp-jbh7Yo8Qo+-w@mail.gmail.com>

> yes indeed, please find attached.

Thanks. Here some comments on the patch:

- It seems you use a table name (and schema) for a key to identify the
  TableMutationEntry and other objects. I think you should use table
  oids for the key because the same table name could exists in
  different schema. Moreover, if the database is different from the
  database when the map entry was created, a map look up could return
  incorrect result. In summary the key should be table oid and
  database oid (which is already done by query cache subsystem).

- In the patch spin lock primitives are introduced. Why can't we use
  semaphore instead? A spin lock uses busy loop, which could increase
  the system load if the duration of locking becomes longer.

- What would happen if the leader watchdog fails and other watchdog
  node take the place of the leader role?

- pool_memory_map_get_ttl() and pool_memory_map_get_stats() are
  defined but are not used anywhere. Why do you have them?

- I think "memory_map" is a too generic name. Can we use more specific
  name for the feature?

Best regards,
--
Tatsuo Ishii
SRA OSS K.K.
English: http://www.sraoss.co.jp/index_en/
Japanese:http://www.sraoss.co.jp

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: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: Proposal: Recent mutated table tracking in memory
  In-Reply-To: <[email protected]>

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

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