RE: Documentation to upgrade logical replication cluster

Поиск
Список
Период
Сортировка
От Hayato Kuroda (Fujitsu)
Тема RE: Documentation to upgrade logical replication cluster
Дата
Msg-id TY3PR01MB988921AFB207C42F354EDA79F57C2@TY3PR01MB9889.jpnprd01.prod.outlook.com
обсуждение исходный текст
Ответ на Re: Documentation to upgrade logical replication cluster  (vignesh C <vignesh21@gmail.com>)
Ответы Re: Documentation to upgrade logical replication cluster
Список pgsql-hackers
Dear Vignesh,

Thanks for updating the patch! Here are my comments for v6.

01.
```
+   <glossterm>Logical replication cluster</glossterm>
+   <glossdef>
+    <para>
+     A set of publisher and subscriber instance with publisher instance
+     replicating changes to the subscriber instance.
+    </para>
+   </glossdef>
```

Should we say 1:N relationship is allowed?

02.
```
@@ -70,6 +70,7 @@ PostgreSQL documentation
    pg_upgrade supports upgrades from 9.2.X and later to the current
    major release of <productname>PostgreSQL</productname>, including snapshot and beta releases.
   </para>
+
  </refsect1>
```

Unnecessary blank.

03.
```
   <para>
-   These are the steps to perform an upgrade
-   with <application>pg_upgrade</application>:
+   Below are the steps to perform an upgrade
+   with <application>pg_upgrade</application>.
   </para>
```

I'm not sure it should be included in this patch.

04.
```
+       If the old primary is prior to version 17.0, then no slots on the primary
+       are copied to the new standby, so all the slots on the old standby must
+       be recreated manually.
```

I think that "all the slots on the old standby" must be created manually in any
cases. Therefore, the preposition ", so" seems not correct.

05.
```
If the old primary is version 17.0 or later, then
+       only logical slots on the primary are copied to the new standby, but
+       other slots on the old standby are not copied, so must be recreated
+       manually.
```

How about replacing this paragraph to below?

```
All the slots on the old standby must be recreated manually. If the old primary
is version 17.0 or later, then only logical slots on the primary are copied to the
new standby.
```

Best Regards,
Hayato Kuroda
FUJITSU LIMITED
https://www.fujitsu.com/ 


В списке pgsql-hackers по дате отправления:

Предыдущее
От: Michael Paquier
Дата:
Сообщение: Re: int4->bool test coverage
Следующее
От: Peter Smith
Дата:
Сообщение: Re: src/bin/pg_upgrade/t/004_subscription.pl test comment fix