> On Thu, Feb 6, 2014 at 3:42 PM, Kyotaro HORIGUCHI > <horiguchi.kyot...@lab.ntt.co.jp> wrote: >> Hello, I've understood how this works and seems working as >> expected. >> >> >> The orphan section handles on postmaster have become a matter of >> documentation.
I had explained this in function header of dsm_keep_segment(). >> Besides all above, I'd like to see a comment for the win32 code >> about the 'DuplicateHandle hack', specifically, description that >> the DuplicateHandle pushes the copy of the section handle to the >> postmaster so the section can retain for the postmaster lifetime. I had added a new function in dsm_impl.c for platform specific handling and explained about new behaviour in function header. > - "Global/PostgreSQL.%u" is the same literal constant with that > occurred in dsm_impl_windows. It should be defined as a > constant (or a macro). Changed to hash define. > - dms_impl_windows uses errcode_for_dynamic_shared_memory() for > ereport and it finally falls down to > errcode_for_file_access(). I think it is preferable, maybe Changed as per suggestion. Please find new version of patch attached with this mail containing above changes. Thanks for review. With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com
dsm_keep_segment_v3.patch
Description: Binary data
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers