Обсуждение: Patch for reorderbuffer.c documentation.

Поиск
Список
Период
Сортировка

Patch for reorderbuffer.c documentation.

От
Dave Cramer
Дата:
This started out with just fixing 

"One option do deal" to " One option to deal"

But after reading the rest I'd propose the following patch.

Dave Cramer
Вложения

Re: Patch for reorderbuffer.c documentation.

От
"David G. Johnston"
Дата:
On Fri, Jul 17, 2020 at 8:10 AM Dave Cramer <davecramer@gmail.com> wrote:
This started out with just fixing 

"One option do deal" to " One option to deal"

But after reading the rest I'd propose the following patch.

Suggest replacing "though" with "however" instead of trying to figure out what amount of commas is readable (the original seemed better IMO).

"However, the transaction records are fairly small and"

The rest is straight-forward.

David J.

Re: Patch for reorderbuffer.c documentation.

От
Dave Cramer
Дата:


On Fri, 17 Jul 2020 at 11:17, David G. Johnston <david.g.johnston@gmail.com> wrote:
On Fri, Jul 17, 2020 at 8:10 AM Dave Cramer <davecramer@gmail.com> wrote:
This started out with just fixing 

"One option do deal" to " One option to deal"

But after reading the rest I'd propose the following patch.

Suggest replacing "though" with "however" instead of trying to figure out what amount of commas is readable (the original seemed better IMO).

"However, the transaction records are fairly small and"

Works for me.

Thanks,
Dave

Re: Patch for reorderbuffer.c documentation.

От
Amit Kapila
Дата:
On Fri, Jul 17, 2020 at 8:59 PM Dave Cramer <davecramer@gmail.com> wrote:
>
> On Fri, 17 Jul 2020 at 11:17, David G. Johnston <david.g.johnston@gmail.com> wrote:
>>
>> On Fri, Jul 17, 2020 at 8:10 AM Dave Cramer <davecramer@gmail.com> wrote:
>>>
>>> This started out with just fixing
>>>
>>> "One option do deal" to " One option to deal"
>>>
>>> But after reading the rest I'd propose the following patch.
>>
>>
>> Suggest replacing "though" with "however" instead of trying to figure out what amount of commas is readable (the
originalseemed better IMO).
 
>>
>> "However, the transaction records are fairly small and"
>
>
> Works for me.
>

Thanks, pushed.

-- 
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com