On Mon, Nov 18, 2019 at 5:20 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > > On Thu, Oct 3, 2019 at 4:48 PM Amit Khandekar <amitdkhan...@gmail.com> wrote: > > > > On Wed, 18 Sep 2019 at 12:24, Amit Khandekar <amitdkhan...@gmail.com> wrote: > > > Probably, for now at least, what everyone seems to agree is to take my > > > earlier attached patch forward. > > > > > > I am going to see if I can add a TAP test for the patch, and will add > > > the patch into the commitfest soon. > > > > Attached is an updated patch v2. > > > > I see that you have made changes in ReorderBufferRestoreChanges to use > PathNameOpenFile, but not in ReorderBufferSerializeTXN. Is there a > reason for the same? >
I have one more question regarding this patch. It seems to me that the files opened via OpenTransientFile or OpenTemporaryFile are automatically closed at transaction end(abort), but that doesn't seem to be the case for files opened with PathNameOpenFile. See AtEOXact_Files and AtEOSubXact_Files. So, now with the change proposed by this patch, don't we need to deal it in some other way? -- With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com