Thank you for reviewing!
On Thu, Mar 8, 2018 at 6:07 PM, Ildar Musin <i.musin@postgrespro.ru> wrote:
> Just couple remarks. I would rename 'requested' variable in
> AutoVacuumRequestWork() func to something like 'success' or 'result'.
> Because request is something caller does. And I would also rephrase log
> message as follows:
>
> request for autovacuum work item "%s" for "%s" failed
Agreed.
On Thu, Mar 8, 2018 at 10:46 PM, Alvaro Herrera
<alvherre@2ndquadrant.com> wrote:
> Hi
>
> I was thinking that the BRIN code requesting the workitem would print
> the error message based on the return value. There is no point to
> returning a boolean indicator if the caller isn't going to do anything
> with it ... This means you don't need to convert the type to string in
> autovacuum.c (which would defeat attempts at generalizing this code).
>
Agreed.
Attached an updated patch. Please review it.
Regards,
--
Masahiko Sawada
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center