Re: Pgoutput not capturing the generated columns

On 21.01.25 09:27, vignesh C wrote:
>> Maybe I have some fundamental misunderstanding here, but I don't see
>> why "this approach cannot be used with psql because older version
>> servers may not have these columns". Not having the columns is the
>> whole point of using an alias approach in the first place e.g. the
>> below table t1 does not have a column called "banana" but it works
>> just fine to select an alias using that name...
> 
> This is simpler than maintaining the indexes. I misunderstood your
> comment to include displaying the columns for older versions, I did
> not want to display a value for the older versions as these columns do
> not exist. I have updated the patch based on the alias approach. The
> attached patch has the changes for the same.

The v54-0004 patch looks ok to me.  Thanks for working on this.

I will wait until that gets committed before moving forward with the 
virtual generated columns patch.


A few quick comments on the v54-0005 patch (the documentation one):

+<programlisting>
+test_pub=# CREATE TABLE tab_gen_to_gen (a int, b int GENERATED ALWAYS 
AS (a + 1) STORED);

I don't like when examples include the prompt like this.  Then it's 
harder to copy the commands out of the examples.

+     <row>
+ 
<entry>No</entry><entry>GENERATED</entry><entry>GENERATED</entry><entry>Publisher 
table column is not replicated. Use the subscriber table generated 
column value.</entry>
+     </row>

This should be formatted vertically, one line per <entry>.

+<programlisting>
+test_pub=# CREATE TABLE t1 (a int PRIMARY KEY, b int,
+test_pub(#                  c int GENERATED ALWAYS AS (a + 1) STORED,
+test_pub(#                  d int GENERATED ALWAYS AS (b + 1) STORED);

Also here check that this is formatted more consistently with the rest 
of the documentation.

+  <itemizedlist>
+   <listitem><para>
+    <literal>t1.a</literal> is a regular column. It gets replicated 
normally.
+   </para></listitem>
+   <listitem><para>
+    <literal>t1.b</literal> is a regular column. It gets replicated 
normally.
+   </para></listitem>

Leave more whitespace.  Typically, there is a blank line between block 
elements on the same level.