pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  

help / color / mirror / Atom feed
From: Jesper Pedersen <jesper.pedersen@redhat.com>
To: Thomas Munro <thomas.munro@enterprisedb.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Magnus Hagander <magnus@hagander.net>
Cc: Oskari Saarenmaa <os@ohmu.fi>
Cc: Pg Hackers <pgsql-hackers@postgresql.org>
Cc: Tobias Oberstein <tobias.oberstein@gmail.com>
Subject: Re: pread() and pwrite()
Date: Mon, 5 Nov 2018 08:44:34 -0500
Message-ID: <13bf4246-061f-6187-e881-547a82379f70@redhat.com> (raw)
In-Reply-To: <CAEepm=3QOz_BEP9tz66q6tXg=dKkXCnkzo_VcFttajgpTNTS+g@mail.gmail.com>
References: <CAEepm=02rapCpPR3ZGF2vW=SBHSdFYO_bz_f-wwWJonmA3APgw@mail.gmail.com>
	<20180720081459.GC22214@saarenmaa.fi>
	<CAEepm=2qRXWM+uX26dPn6S=TAqi8FfuAZ9vG8tZW9Jo7w6YeQw@mail.gmail.com>
	<7f379440-dfec-70ca-8a0d-942f1381936b@redhat.com>
	<c2f56d0a-cadd-3df1-ae48-b84dc8128c37@redhat.com>
	<CAEepm=1DZSzdBRDbXFXSiu_Hiv08f22kFmgCi7ZBRERF_=ZCPg@mail.gmail.com>
	<983a8d48-64fa-e2f7-55c4-41e20bc51ef5@redhat.com>
	<CAEepm=15aoFSr9G25T4wP_6vTKd1bPAViu5pZtrObyviMvowpQ@mail.gmail.com>
	<19344.1539050134@sss.pgh.pa.us>
	<CAEepm=1FDQOTtpSDP+VX15t1F+cg1jOHKNs-NNM=qFYYk_0++w@mail.gmail.com>
	<21196.1539053390@sss.pgh.pa.us>
	<2145.1539058113@sss.pgh.pa.us>
	<CAEepm=1X9htKVEXbPZnH8ebF+Mo5kVkTZq3pg9gmdu=1Xx8rnA@mail.gmail.com>
	<4cf40b3d-96ed-7597-fb42-e5e29c5a40fe@redhat.com>
	<CAEepm=0gKm3WvzUKkfynkc6TFJ3MrfmX7MBFi9nPvDd3gTapOw@mail.gmail.com>
	<CAEepm=3QOz_BEP9tz66q6tXg=dKkXCnkzo_VcFttajgpTNTS+g@mail.gmail.com>

Hi Thomas,

On 11/5/18 7:08 AM, Thomas Munro wrote:
> On Sun, Nov 4, 2018 at 12:03 AM Thomas Munro
> <thomas.munro@enterprisedb.com> wrote:
>> On Sat, Nov 3, 2018 at 2:07 AM Jesper Pedersen
>> <jesper.pedersen@redhat.com> wrote:
>>> This still applies, and passes make check-world.
>>>
>>> I wonder what the commit policy is on this, if the Windows part isn't
>>> included. I read Heikki's comment [1] as it would be ok to commit
>>> benefiting all platforms that has pread/pwrite.
>>
>> Here's a patch to add Windows support by supplying
>> src/backend/port/win32/pread.c.  Thoughts?
> 
> If we do that, I suppose we might as well supply implementations for
> HP-UX 10.20 as well, and then we can get rid of the conditional macro
> stuff at various call sites and use pread() and pwrite() freely.
> Here's a version that does it that way.  One question is whether the
> caveat mentioned in patch 0001 is acceptable.
> 

Passed check-world, but I can't verify the 0001 patch. Reading the the 
API it looks ok to me.

I guess the caveat in 0001 is ok, as it is a side-effect of the 
underlying API.

Best regards,
  Jesper




view thread (41+ messages)  latest in thread

Message-ID: <13bf4246-061f-6187-e881-547a82379f70@redhat.com>
Permalink:  ../13bf4246-061f-6187-e881-547a82379f70@redhat.com/
Also on:    postgresql.org/message-id/13bf4246-061f-6187-e881-547a82379f70@redhat.com

 · 

reply
Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-hackers@postgresql.org
  Cc: jesper.pedersen@redhat.com, thomas.munro@enterprisedb.com, tgl@sss.pgh.pa.us, magnus@hagander.net, os@ohmu.fi, tobias.oberstein@gmail.com
  Subject: Re: pread() and pwrite()
  In-Reply-To: <13bf4246-061f-6187-e881-547a82379f70@redhat.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox