On Wed, Mar 6, 2019 at 6:53 PM Michael Paquier <mich...@paquier.xyz> wrote: > On Wed, Mar 06, 2019 at 02:47:16PM +0000, Robert Haas wrote: > > Removed unused variable, openLogOff. > > Is that right for the report if data is written in chunks? The same > patch has been proposed a couple of weeks ago, and I commented about > it as follows: > https://www.postgresql.org/message-id/20190129043439.gb3...@paquier.xyz
Oh, sorry, I didn't see the earlier thread. You're right that this is messed up: if we're going to report startOffset rather than openLogOff, we need to report nleft rather than bytes. I would prefer to change nbytes -> nleft rather than anything else, though, because it seems to me that we should report the offset and length that actually failed, not the start of the whole chunk which partially succeeded. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
