From: | Boszormenyi Zoltan <zb(at)cybertec(dot)at> |
---|---|
To: | Antonin Houska <antonin(dot)houska(at)gmail(dot)com> |
Cc: | Gibheer <gibheer(at)zero-knowledge(dot)org>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Backup throttling |
Date: | 2013-08-19 17:04:07 |
Message-ID: | 52125007.40201@cybertec.at |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
2013-07-31 22:50 keltezéssel, Antonin Houska írta:
> On 07/31/2013 07:13 AM, Gibheer wrote:
>> Hi,
>>
>> That is a really nice feature.
> I don't pretend it's my idea, I just coded it. My boss proposed the feature as such :-)
>> I took a first look at your patch and some empty lines you added (e.g. line 60 your patch). Can you remove
>> them?
> Sure, will do in the next version.
>> Why did you move localGetCurrentTimestamp() into streamutil.c? Is sys/time.h still needed in receivelog.c after the move?
> Because both receivelog.c and pg_basebackup.c need it now. I thought I could move
> localTimestampDifference() and localTimestampDifferenceExceeds() as well for the sake of
> consistency (these are actually utilities too) but I didn't get convinced enough that
> the feature alone justifies such a change.
>
> As mentioned in
> http://www.postgresql.org/message-id/20130731173624.GX14652@eldon.alvh.no-ip.org these
> functions ideally shouldn't have separate implementation at all. However the problem is
> that pg_basebackup is not linked to the backend.
Have you considered the src/common directory?
>
> You're right about sys/time.h, it's included via via streamutil.h. I'll fix that too.
>> I will try your patch later today to see, if it works.
>>
> Whenever you have time. Thanks!
>
> // Tony
Best regards,
Zoltán Böszörményi
--
----------------------------------
Zoltán Böszörményi
Cybertec Schönig & Schönig GmbH
Gröhrmühlgasse 26
A-2700 Wiener Neustadt, Austria
Web: http://www.postgresql-support.de
http://www.postgresql.at/
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2013-08-19 17:20:12 | Re: Backup throttling |
Previous Message | Alvaro Herrera | 2013-08-19 16:57:44 | Re: [BUGS] 9.3beta2: Failure to pg_upgrade |