On Wed, Mar 10, 2021 at 1:27 PM Jeevan Ladhe <jeevan.la...@enterprisedb.com> wrote: > > On Wed, Mar 10, 2021 at 10:44 AM Bharath Rupireddy > <bharath.rupireddyforpostg...@gmail.com> wrote: >> >> Hi, >> >> While providing thoughts on [1], I observed that the error messages >> that are emitted while adding foreign, temporary and unlogged tables >> can be improved a bit from the existing [2] to [3]. > > +1 for improving the error messages here.
Thanks for taking a look at the patch. >> Attaching a small patch. Thoughts? > > I had a look at the patch and it looks good to me. However, I think after > you have added the specific kind of table type in the error message itself, > now the error details seem to be giving redundant information, but others > might > have different thoughts. The error detail is to give a bit of information of what and all relation types are unsupported with the create publication statement. But with the error message now showing up the type of relation, the detail message looks redundant to me as well. If agreed, I can remove that. Thoughts? > The patch itself looks good otherwise. Also the make check and postgres_fdw > check looking good. Thanks. With Regards, Bharath Rupireddy. EnterpriseDB: http://www.enterprisedb.com
