On 21/12/16 04:31, Craig Ringer wrote:
> On 20 December 2016 at 15:03, Petr Jelinek <petr.jeli...@2ndquadrant.com> 
> wrote:
> 
>> But in 0003 I don't understand following code:
>>> +     if (endpos != InvalidXLogRecPtr && !do_start_slot)
>>> +     {
>>> +             fprintf(stderr,
>>> +                             _("%s: cannot use --create-slot or 
>>> --drop-slot together with --endpos\n"),
>>> +                             progname);
>>> +             fprintf(stderr, _("Try \"%s --help\" for more 
>>> information.\n"),
>>> +                             progname);
>>> +             exit(1);
>>> +     }
>>
>> Why is --create-slot and --endpos not allowed together?
> 
> Actually, the test is fine, the error is just misleading due to my
> misunderstanding.
> 
> The fix is simply to change the error message to
> 
>                                 _("%s: --endpos may only be specified
> with --start\n"),
> 
> so I won't post a separate followup patch.
> 

Ah okay makes sense. The --create-slot + --endpos should definitely be
allowed combination, especially now that we can extend this to
optionally use temporary slot.

-- 
  Petr Jelinek                  http://www.2ndQuadrant.com/
  PostgreSQL Development, 24x7 Support, Training & Services


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to