On Tue, Nov 13, 2018 at 11:32 AM Haribabu Kommi <kommi.harib...@gmail.com> wrote: > > On Mon, Nov 12, 2018 at 6:34 PM Amit Kapila <amit.kapil...@gmail.com> wrote: >> > I can revert it back to void, >> > >> >> +1, as we don't see any good reason to break backward compatibility. > > > Thanks for the review. > Attached the updated patch with return type as void. >
With this patch, we are intending to remove the entries matching userid, dbid, queryid from hash table (pgss_hash), but the contents of the file ( pgss_query_texts.stat) will remain unchanged unless all of the entries are removed from hash table. This appears fine to me, especially because there is no easy way to remove the contents from the file. Does anybody see any problem with this behavior? -- With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com
