From: | "wangw(dot)fnst(at)fujitsu(dot)com" <wangw(dot)fnst(at)fujitsu(dot)com> |
---|---|
To: | Önder Kalacı <onderkalaci(at)gmail(dot)com>, Peter Smith <smithpb2250(at)gmail(dot)com> |
Cc: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | RE: [PATCH] Use indexes on the subscriber when REPLICA IDENTITY is full on the publisher |
Date: | 2022-09-22 03:36:02 |
Message-ID: | OS3PR01MB6275DF45FF5BB98A548CE80F9E4E9@OS3PR01MB6275.jpnprd01.prod.outlook.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tues, Sep 20, 2022 at 18:30 PM Önder Kalacı <onderkalaci(at)gmail(dot)com> wrote:
> Thanks for the reviews, attached v12.
Thanks for your patch. Here is a question and a comment:
1. In the function GetCheapestReplicaIdentityFullPath.
+ if (rel->pathlist == NIL)
+ {
+ /*
+ * A sequential scan could have been dominated by by an index scan
+ * during make_one_rel(). We should always have a sequential scan
+ * before set_cheapest().
+ */
+ Path *seqScanPath = create_seqscan_path(root, rel, NULL, 0);
+
+ add_path(rel, seqScanPath);
+ }
This is a question I'm not sure about:
Do we need this part to add sequential scan?
I think in our case, the sequential scan seems to have been added by the
function make_one_rel (see function set_plain_rel_pathlist). If I am missing
something, please let me know. BTW, there is a typo in above comment: `by by`.
2. In the file execReplication.c.
+#ifdef USE_ASSERT_CHECKING
+#include "catalog/index.h"
+#endif
#include "commands/trigger.h"
#include "executor/executor.h"
#include "executor/nodeModifyTable.h"
#include "nodes/nodeFuncs.h"
#include "parser/parse_relation.h"
#include "parser/parsetree.h"
+#ifdef USE_ASSERT_CHECKING
+#include "replication/logicalrelation.h"
+#endif
I think it's fine to only add `logicalrelation.h` here, because `index.h` has
been added by `logicalrelation.h`.
Regards,
Wang wei
From | Date | Subject | |
---|---|---|---|
Next Message | Justin Pryzby | 2022-09-22 03:37:16 | Re: pg_basebackup's --gzip switch misbehaves |
Previous Message | wangw.fnst@fujitsu.com | 2022-09-22 03:29:13 | RE: Perform streaming logical transactions by background workers and parallel apply |