Horton

Thread

Commits

  1. Fix statenames in mergejoin comments

  1. doc: join state for merge join

    Zhihong Yu <zyu@yugabyte.com> — 2022-02-19T13:20:14Z

    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

  2. Re: doc: join state for merge join

    Daniel Gustafsson <daniel@yesql.se> — 2022-02-19T19:24:09Z

    > 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/