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: Tue, 03 Feb 2026 16:43:53 +0900 (JST)
Message-ID: <[email protected]> (raw)
In-Reply-To: <CACeKOO1W=NxxqxBFtVa++skatF67DawxJwF5R6iMnqYw=OHoHA@mail.gmail.com>
References: <CACeKOO36wVT2k5b9G2J4XYvjNQTYf-ARoh8Lp-jbh7Yo8Qo+-w@mail.gmail.com>
	<[email protected]>
	<CACeKOO1W=NxxqxBFtVa++skatF67DawxJwF5R6iMnqYw=OHoHA@mail.gmail.com>

Hi Nadav,

Thank you for updating the patch!

> Thank you for the comments!
> 
> I agree with all of them. Let me know what you think of the changes and new
> naming.

I still think "memory_map" is too generic. Anything put on memory for
data mapping could be called "memory map". I recommend to change the
name to more feature specific one: What about replacing "memory_map"
with "track_table_mutation"? It's a little bit longer name but it
clearly represents the feature. Any better ideas are welcome.

- memory_map_enabled: Enable/disable the feature (default: off)
- memory_map_ttl_factor: TTL multiplier for replication delay (default: 5.0)
- memory_map_cold_start_duration: Cold start period in ms (default: 2000)
- memory_map_table_buckets: Hash buckets for table map (default: 1024)
- memory_map_table_size: Max tracked tables (default: 2048)
- memory_map_query_buckets: Hash buckets for query cache (default: 2048)
- memory_map_query_cache_size: Max cached queries (default: 10000)

Also I feel memory_map_query_cache_size is confusing because there's
already "query cache" feature in pgpool.  Can we change it something
like "query_parse_cache_size"?

Review comments:

(1) Why the regression test is 45? Shouldn't it be 42? (the last
feature test is 041.external_replication_delay).

(2) You enhance the patch to deal with leader watch changing. That's
good. However, I don't see a test case for it in test.sh.

(3) It seems the patch does not support TRUNCATE, MERGE, PREPARE and
WITH + updating. If so, it should be noted in the docs as a limitation
of the feature.
--
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