Hi Nadav,

> Hi Tatsuo,
>
> Thank you for the thorough review and the fix to the tests!. Here's the
> updated patch addressing all your comments.
>
> re - replication_delay_source_cmd requirement
>
> Good catch ― the feature now also works when `delay_threshold_by_time > 0`.
> I've added the TTL update call to `check_replication_time_lag()` (the
> pg_stat_replication path), not just
> `check_replication_time_lag_with_cmd()`. The docs are updated to reflect
> that either `replication_delay_source_cmd` or `delay_threshold_by_time` can
> provide the time-based delay.
>
> re - Documentation compile error
>
> Fixed ― the xref was pointing to `runtime-config-track-table-mutation` but
> the actual section ID is `runtime-config-table-mutation-map`.
>
> Thanks again and looking forward to hearing back from you.

While looking into your patch, I noticed that following part would be
better to be committed as a separate patch as it's actually a bug fix,
not related to the feature: query cache is not invalidated if MERGE
statement is executed. I will take care of it and get back to you.

--- a/src/query_cache/pool_memqcache.c
+++ b/src/query_cache/pool_memqcache.c
@@ -1305,6 +1305,12 @@ pool_extract_table_oids(Node *node, int **oidsp)
                }
                return num_oids;
        }
+       else if (IsA(node, MergeStmt))
+       {
+               MergeStmt  *stmt = (MergeStmt *) node;
+
+               table = make_table_name_from_rangevar(stmt->relation);
+       }
        else if (IsA(node, ExplainStmt))
        {
                ListCell   *cell;

Thanks for letting me know about the bug!

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