Thread: doc: join state for merge join

doc: join state for merge join

From
Zhihong Yu
Date:
Hi,
When i was looking at the code for ExecMergeJoin(), I noticed a few places where the join state name in the comment doesn't match the actual state.

Here is a small patch with correction.

Cheers
Attachment

Re: doc: join state for merge join

From
Daniel Gustafsson
Date:
> On 19 Feb 2022, at 14:20, Zhihong Yu <zyu@yugabyte.com> wrote:

> When i was looking at the code for ExecMergeJoin(), I noticed a few places where the join state name in the comment
doesn'tmatch the actual state. 

Given the comments on the other states, I'm bound to agree.  Unless there are
objections I'll go ahead with this one.

--
Daniel Gustafsson        https://vmware.com/