doc: join state for merge join

Started by Zhihong Yuabout 4 years ago2 messageshackers
Jump to latest
#1Zhihong Yu
zyu@yugabyte.com

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

Attachments:

merge-join-state-doc.patchapplication/octet-stream; name=merge-join-state-doc.patchDownload+3-3
#2Daniel Gustafsson
daniel@yesql.se
In reply to: Zhihong Yu (#1)
Re: doc: join state for merge join

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't match 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/