Thread: Patch for reorderbuffer.c documentation.

Patch for reorderbuffer.c documentation.

From
Dave Cramer
Date:
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
Attachment

Re: Patch for reorderbuffer.c documentation.

From
"David G. Johnston"
Date:
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.

From
Dave Cramer
Date:


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.

From
Amit Kapila
Date:
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