Re: Patch: Range Merge Join

From: Thomas <thomasmannhart97(at)gmail(dot)com>
To: daniel(at)yesql(dot)se
Cc: Jaime Casanova <jcasanov(at)systemguards(dot)com(dot)ec>, pgsql <pgsql(at)j-davis(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org, dignoes(at)inf(dot)unibz(dot)it, boehlen(at)ifi(dot)uzh(dot)ch, p(dot)moser(at)noi(dot)bz(dot)it, gamper(at)inf(dot)unibz(dot)it, Thomas Mannhart <thomas_m(at)hotmail(dot)ch>
Subject: Re: Patch: Range Merge Join
Date: 2021-11-17 14:45:26
Message-ID: CAMWfgiBRETALLvhk9P3dCsii8cL1XyYHv87=MH4OtnT5Y+o9Ng@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Thank you for the feedback and sorry for the oversight. I fixed the bug and
attached a new version of the patch.

Kind Regards, Thomas

Am Mi., 17. Nov. 2021 um 15:03 Uhr schrieb Daniel Gustafsson <
daniel(at)yesql(dot)se>:

> This patch fails to compile due to an incorrect function name in an
> assertion:
>
> nodeMergejoin.c:297:9: warning: implicit declaration of function
> 'list_legth' is invalid in C99 [-Wimplicit-function-declaration]
> Assert(list_legth(node->rangeclause) < 3);
> ^
>
> --
> Daniel Gustafsson https://vmware.com/
>
>

Attachment Content-Type Size
postgres-rmj_11-17-21.patch application/octet-stream 47.1 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2021-11-17 14:49:58 Re: Propose a new hook for mutating the query bounds
Previous Message vignesh C 2021-11-17 14:42:44 Re: Printing backtrace of postgres processes