On Thu, Jan 20, 2022 at 7:51 AM Peter Smith <smithpb2...@gmail.com> wrote: > > 5. DEBUG level 3 > > I found there are 3 debug logs in this patch and they all have DEBUG3 level. > > IMO it is probably OK as-is, >
+1. > but just a comparison I noticed that the > most detailed logging for logical replication worker.c was DEBUG2. > Perhaps row-filter patch should be using DEBUG2 also? > OTOH, the other related files like reorderbuffer.c and snapbuild.c are using DEBUG3 for the detailed messages. So, I think it is probably okay to retain logs as is. -- With Regards, Amit Kapila.
