On Thu, Sep 23, 2021 at 10:33 PM Tomas Vondra <tomas.von...@enterprisedb.com> wrote: > > Hi, > > I finally had time to take a closer look at the patch again, so here's > some review comments. The thread is moving fast, so chances are some of > the comments are obsolete or were already raised in the past. > ... > 11) extra (unnecessary) parens in the deparsed expression > > test=# alter publication p add table t where ((b < 100) and (c < 100)); > ALTER PUBLICATION > test=# \dRp+ p > Publication p > Owner | All tables | Inserts | Updates | Deletes | Truncates | Via root > -------+------------+---------+---------+---------+-----------+---------- > user | f | t | t | t | t | f > Tables: > "public.t" WHERE (((b < 100) AND (c < 100))) >
Euler's fix for this was integrated into v45 [1] ------ [1] https://www.postgresql.org/message-id/CAFPTHDYB4nbxCMAFQGowJtDf7E6uBc%3D%3D_HupBKy7MaMhM%2B9QQA%40mail.gmail.com Kind Regards, Peter Smith. Fujitsu Australia
